[llvm] [libc++] Fix CODEOWNERS file for libc++ (PR #65344)

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 8 00:03:27 PDT 2023


joker-eph wrote:

>  I might be wrong but I would think that being notified a few seconds after when the appropriate tag is added to the PR via Github Actions is enough?

The motivation was to have a "good" notification, that is an email which is directly actionable and not "just a link", which led us to the CODE_OWNERS file.
But Tom figured out some magic to get this done now!

I'd note that LLVM as a project does not have concept of code owners required approval, commit access is technically enough to review and push code, and folks are supposed to apply best judgement with respect to their level of familiarity with parts of the codebase! I'm not sure there is another doc for the project than https://llvm.org/docs/CodeReview.html ?

https://github.com/llvm/llvm-project/pull/65344


More information about the llvm-commits mailing list