[PATCH] D97389: added section on CI system

[Azharuddin Mohammed via Phabricator via llvm-commits]

azharudd added inline comments.


================
Comment at: llvm/docs/DeveloperPolicy.rst:651-652
+
+The main continuous integration (CI) tool for the LLVM project are the 
+`BuildBots <https://lab.llvm.org/buildbot/>`_. These use different *builders* to
+cover a wide variety of sub-projects and configurations. The builds are executed
----------------



================
Comment at: llvm/docs/DeveloperPolicy.rst:657
+
+The BuildBots track the commits on the main branch and the release branches. 
+This means that patches are built and tested after they are merged to the these
----------------



================
Comment at: llvm/docs/DeveloperPolicy.rst:691-697
+There are other CI and testing systems provided by the community, however it is
+optional to monitor these:
+
+* `Green Dragon <http://green.lab.llvm.org/green/>`_
+* `LNT <http://lnt.llvm.org/>`_ performance tracking system
+* `gn bot <http://45.33.8.238/>`_
+* `pre-merge testing <https://github.com/google/llvm-premerge-checks/>`_
----------------
I think it is best to just remove this part.
 - Issues reported by any CI system are to be fixed
 - LNT is not CI, whereas this section is talking about CI system
 - pre-merge testing is optional, covers only Phab reviews and already posts updates on the review


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97389