[PATCH] D61267: Update Phabricator.rst

[Wink Saville via Phabricator via llvm-commits]

winksaville added inline comments.


================
Comment at: llvm/docs/Phabricator.rst:69-70
+runs it will use your editor, set-local-editor_, so you can supply ``Reviewers``
+and ``Subscribers`` in your commit message. It requires **one or more** Reviewers
+and allows **zero or more** Subscribers, finding-reviewer-names_.
+
----------------
lebedev.ri wrote:
> It really should be emphasized that either the appropriate repo field should be set,
> or appropriate -commits list be subscribed, from the start.
I'm don't know what you mean by 'appropriate repo field'. I've emphasied there must be **one or more** Reviewers.
I could explicitly say that Differential is automatically adding "llvm-commits" to Subscribers.

Or do you mean something else?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61267