[PATCH] D61267: Update Phabricator.rst

[Wink Saville via Phabricator via llvm-commits]

winksaville added inline comments.


================
Comment at: llvm/docs/Phabricator.rst:168
 
-* Use ``svn blame`` and the commit log to find names of people who have
+* Use ``git blame`` and the commit log to find names of people who have
   recently modified the same area of code that you are modifying.
----------------
hintonda wrote:
> You could also mention `arc cover` here.
I had it previously, but then felt it needed more explanation, so was lazy and removed it. But it does reasonable information, I'll add it back with an example section.


================
Comment at: llvm/docs/Phabricator.rst:197
+
+The names for Reviewers and Subscribers is the registered Phabricator name for
+the individual. Type the persons "real name" or a portion of the persons "real name" or
----------------
hintonda wrote:
> Cut-n-paste error?
Yes :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61267/new/

https://reviews.llvm.org/D61267