[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