[PATCH] D112755: [docs] Expand a bit on the basics of a buildbot policy

Thu Oct 28 18:25:54 PDT 2021

reames added inline comments.

================
Comment at: llvm/docs/HowToAddABuilder.rst:50
+Most issues with a buildbot should be addressed directly with a bot owner
+via email.  Please CC `Galina Kistanova <mailto:gkistanova at gmail.com>`_.

----------------
mehdi_amini wrote:
> reames wrote:
> > gkistanova wrote:
> > > mehdi_amini wrote:
> > > > We should document how to find this information. Some colleagues somehow couldn't find it in the web interface so it may no be obvious to everyone.
> > > This?
> > > https://llvm.org/docs/DeveloperPolicy.html#working-with-the-ci-system
> > That snippet really seems like it should be out of line with the developer docs.  Would moving the content below "If a build/worker is permanently broken" into this page, and linking from that one seem reasonable?  
> > 
> > (I'll do it in a separate review for sanity check, just checking big picture feasibility.)
> Nice snippet Galina! This is exactly what I had in mind.
> 
> Moving it here and linking from the dev doc to here seems fine to me as well.
https://reviews.llvm.org/D112775

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112755