[PATCH] D97389: added section on CI system

Christian Kühnel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 21 00:36:35 PDT 2021


kuhnel marked 3 inline comments as done.
kuhnel added inline comments.


================
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/>`_
----------------
azharudd wrote:
> 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
you're right, this does not really add value here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97389



More information about the llvm-commits mailing list