[PATCH] D109810: [doc] documentation of code review tools

[Philip Reames via Phabricator via llvm-commits]

reames requested changes to this revision.
reames added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/docs/DeveloperPolicy.rst:131
+
+Here you can find a list of repositories using 
+`Phabricator <https://reviews.llvm.org/>`_ or GitHub Pull Requests for code
----------------
I request a minor reframing here.  Please describe phabricator as the default, and list the github pull requests as the exception to the general policy.  

I'd also leave out any where there's been a mix of both.  We should bias those towards phabricator unless that particular community desires otherwise.  (i.e. contributes their own follow up to list themselves as PRs)


================
Comment at: llvm/docs/DeveloperPolicy.rst:146
+
+**Repositories using Github Pull Requests:**
+
----------------
Order wise, either alphabetical or popularity.  I'd prefer alphabetical.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109810