[llvm] [CI] Make email check workflow fail when author's email is private in Github UI (PR #148694)

David Spickett via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 15 02:07:03 PDT 2025


https://github.com/DavidSpickett commented:

My understanding of this is that before we looked at the commit and if that had a valid email address, that's enough.

The problem is that the email address used when merging is actually governed by GitHub settings not what's stored in git.

So this changes to GraphQL to do the check, but for the PR author, the way to make the email public is the same as before.

Correct?

(I ask you to confirm because at first glance the description feels like 2 layers of settings but in fact, one of them, the actual git commit, is ignored by GitHub)

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


More information about the llvm-commits mailing list