[PATCH] D41956: [docs] Tweak update to Phabricator docs about setting repository for diffs uploaded via web

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 12 07:41:46 PST 2018


asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

I think the change to "To submit an updated patch:" should probably be left out, given we agree there's no benefit currently in specifying the repo for the diff object (as opposed to the review).

With that, looks good to me!



================
Comment at: docs/Phabricator.rst:87
   *Continue*.
-* Leave the Repository and Project fields blank.
+* In the Repository field, enter the name of the project (LLVM, Clang,
+  etc.) to which the review should be sent.
----------------
This change could probably be left out - there's no benefit we can see in setting the repository when posting an updated patch.


https://reviews.llvm.org/D41956





More information about the llvm-commits mailing list