This is an archive of the discontinued LLVM Phabricator instance.

Expand Phabricator docs slightly
ClosedPublic

Authored by scott-0 on Jun 26 2015, 10:21 AM.

Details

Reviewers
scott-0
klimek
Summary

Here's a patch to the Phabricator web UI instructions in the docs that adds a couple missing button clicks and is a somewhat round-about way of asking a couple questions:

  • What is was meant by the pre-exisitng TODO?
  • Should the Repository and Project fields be used?

When I get the answers I'll update the patch. Thanks.

Diff Detail

Event Timeline

scott-0 updated this revision to Diff 28575.Jun 26 2015, 10:21 AM
scott-0 retitled this revision from to Expand Phabricator docs slightly.
scott-0 updated this object.
scott-0 edited the test plan for this revision. (Show Details)
scott-0 added a subscriber: Unknown Object (MLST).

The TODO looks like it has been there since Manuel created the doc in 2012. Probably not even Manuel remembers what he thought was missing, after all this time.

Leaving the Project and Repository blank seems to work pretty well. :-)

docs/Phabricator.rst
63

Maybe "browse to" instead of "choose" (the button does say Browse).
Of course you can always just type in the filename but I never do it that way.

klimek edited edge metadata.Jul 1 2015, 6:05 AM

Thx!

docs/Phabricator.rst
63–65

Ugh, no idea what I thought...

64

We've set up a single large repo, so leaving that is fine.

73

Nope. We only have a single repo.

scott-0 updated this revision to Diff 28862.Jul 1 2015, 6:18 AM
scott-0 edited edge metadata.

Thanks for the reviews; comments addressed.

klimek accepted this revision.Jul 1 2015, 6:24 AM
klimek edited edge metadata.

lg. thx

This revision is now accepted and ready to land.Jul 1 2015, 6:24 AM
scott-0 accepted this revision.Jul 1 2015, 6:42 AM
scott-0 added a reviewer: scott-0.

committed as r241161

klimek closed this revision.Jul 3 2015, 6:56 AM