Skip to content

Commit bf1a554

Browse files
authored
Document the requirement that commits have a public email address (#109318)
See https://discourse.llvm.org/t/hidden-emails-on-github-should-we-do-something-about-it/74223 for details about why this is important to the community. Note, we currently have soft enforcement for this requirement in the form of a bot which posts comments letting patch authors know their email is private, so we're already setting expectations in practice; this PR is documenting those expectations for clarity.
1 parent 0205667 commit bf1a554

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

llvm/docs/DeveloperPolicy.rst

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,13 @@ When submitting patches, please do not add confidentiality or non-disclosure
105105
notices to the patches themselves. These notices conflict with the LLVM
106106
licensing terms and may result in your contribution being excluded.
107107

108+
The LLVM project uses email to communicate to contributors outside of the
109+
GitHub platform about their past contributions. Primarily, our buildbot
110+
infrastructure uses emails to contact contributors about build and test
111+
failures. Therefore, the LLVM community requires contributors to have a public
112+
email address associated with their GitHub commits, so please ensure that "Keep
113+
my email addresses private" is disabled in your
114+
`account settings <https://github.com/settings/emails>`_.
108115

109116
.. _code review:
110117

0 commit comments

Comments
 (0)