This is an archive of the discontinued LLVM Phabricator instance.

Differential D11527

docs: update arcanist links
ClosedPublic

Authored by martell on Jul 27 2015, 8:09 AM.

Download Raw Diff

Details

Reviewers

yaron.keren

Commits

rG182b4bbc2af5: docs: update arcanist links
rL243408: docs: update arcanist links

Summary

I need a test commit for using arc.
This seems like an appropriate commit to use as a test

We may want to port this commit back to 3.7 also

Diff Detail

Repository: rL LLVM

Event Timeline

martell updated this revision to Diff 30696.Jul 27 2015, 8:09 AM

martell retitled this revision from to docs: update arcanist links.

martell updated this object.

martell added a subscriber: llvm-commits.

Updated to use https:// as it redirects to that anyway.

skip a redirect

LGTM
Aside from the test commit, in the future you can fix broken links without review, that's trivial.

This revision is now accepted and ready to land.Jul 27 2015, 8:41 AM

yeah

In D11527#212583, @yaron.keren wrote:

LGTM
Aside from the test commit, in the future you can fix broken links without review, that's trivial.

Yeah I know chris asked me to do a test on something trivial like this
It seems I can't push via arc land via the git repo
must all commits done via svn ?

Closed by commit rL243408: docs: update arcanist links (authored by martell). · Explain WhyJul 28 2015, 4:44 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents

Path

Size

llvm/

trunk/

docs/

Phabricator.rst

4 lines

Diff 30802

llvm/trunk/docs/Phabricator.rst

	Show First 20 Lines • Show All 156 Lines • ▼ Show 20 Lines
	on an `llvm-reviews GitHub project`_ where folks can hack on it and post pull			on an `llvm-reviews GitHub project`_ where folks can hack on it and post pull
	requests. We're looking into what the right long-term hosting for this is, but			requests. We're looking into what the right long-term hosting for this is, but
	note that it is a derivative of an existing open source project, and so not			note that it is a derivative of an existing open source project, and so not
	trivially a good fit for an official LLVM project.			trivially a good fit for an official LLVM project.

	.. _LLVM's Phabricator: http://reviews.llvm.org			.. _LLVM's Phabricator: http://reviews.llvm.org
	.. _`http://reviews.llvm.org`: http://reviews.llvm.org			.. _`http://reviews.llvm.org`: http://reviews.llvm.org
	.. _Code Repository Browser: http://reviews.llvm.org/diffusion/			.. _Code Repository Browser: http://reviews.llvm.org/diffusion/
	.. _Arcanist Quick Start: http://www.phabricator.com/docs/phabricator/article/Arcanist_Quick_Start.html			.. _Arcanist Quick Start: https://secure.phabricator.com/book/phabricator/article/arcanist_quick_start/
	.. _Arcanist User Guide: http://www.phabricator.com/docs/phabricator/article/Arcanist_User_Guide.html			.. _Arcanist User Guide: https://secure.phabricator.com/book/phabricator/article/arcanist/
	.. _llvm-reviews GitHub project: https://github.com/r4nt/llvm-reviews/			.. _llvm-reviews GitHub project: https://github.com/r4nt/llvm-reviews/