[llvm] Document the requirement that commits have a public email address (PR #109318)

Aaron Ballman via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 17 06:11:50 PDT 2024


https://github.com/AaronBallman updated https://github.com/llvm/llvm-project/pull/109318

>From c137dc63c95156e220a4e8474162042f2d5445cf Mon Sep 17 00:00:00 2001
From: Aaron Ballman <aaron at aaronballman.com>
Date: Thu, 19 Sep 2024 14:14:07 -0400
Subject: [PATCH 1/3] Document the requirement that emails be public

See https://discourse.llvm.org/t/hidden-emails-on-github-should-we-do-something-about-it/74223
for details about why this is useful.

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.
---
 llvm/docs/DeveloperPolicy.rst | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index caa4b31b949c92..86335ab8b868fd 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -86,6 +86,11 @@ that notices of confidentiality or non-disclosure cannot be respected.
 Making and Submitting a Patch
 -----------------------------
 
+The LLVM community requires contributors to have a public email address
+associated with their GitHub account. Please ensure that "Keep my email
+addresses private" is disabled in your
+`account settings <https://github.com/settings/emails>`_.
+
 When making a patch for review, the goal is to make it as easy for the reviewer
 to read it as possible.  As such, we recommend that you:
 

>From 12a4a3ce6c25e817a82e8282c06c23857079a2d3 Mon Sep 17 00:00:00 2001
From: Aaron Ballman <aaron at aaronballman.com>
Date: Thu, 19 Sep 2024 14:32:09 -0400
Subject: [PATCH 2/3] Slight wording tweak for clarity

---
 llvm/docs/DeveloperPolicy.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index 86335ab8b868fd..34bcd507c4e579 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -87,7 +87,7 @@ Making and Submitting a Patch
 -----------------------------
 
 The LLVM community requires contributors to have a public email address
-associated with their GitHub account. Please ensure that "Keep my email
+associated with their GitHub commits, so please ensure that "Keep my email
 addresses private" is disabled in your
 `account settings <https://github.com/settings/emails>`_.
 

>From aa4f52e5f0aab1d35defb2ad5719d58f85843060 Mon Sep 17 00:00:00 2001
From: Aaron Ballman <aaron at aaronballman.com>
Date: Thu, 17 Oct 2024 09:11:20 -0400
Subject: [PATCH 3/3] Move documentation around, add more motivation

---
 llvm/docs/DeveloperPolicy.rst | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/llvm/docs/DeveloperPolicy.rst b/llvm/docs/DeveloperPolicy.rst
index 34bcd507c4e579..0ecf1423e60371 100644
--- a/llvm/docs/DeveloperPolicy.rst
+++ b/llvm/docs/DeveloperPolicy.rst
@@ -86,11 +86,6 @@ that notices of confidentiality or non-disclosure cannot be respected.
 Making and Submitting a Patch
 -----------------------------
 
-The LLVM community requires contributors to have a public email address
-associated with their GitHub commits, so please ensure that "Keep my email
-addresses private" is disabled in your
-`account settings <https://github.com/settings/emails>`_.
-
 When making a patch for review, the goal is to make it as easy for the reviewer
 to read it as possible.  As such, we recommend that you:
 
@@ -110,6 +105,13 @@ When submitting patches, please do not add confidentiality or non-disclosure
 notices to the patches themselves.  These notices conflict with the LLVM
 licensing terms and may result in your contribution being excluded.
 
+The LLVM project uses email to communicate to contributors outside of the
+GitHub platform about their past contributions. Primarily, our buildbot
+infrastructure uses emails to contact contributors about build and test
+failures. Therefore, the LLVM community requires contributors to have a public
+email address associated with their GitHub commits, so please ensure that "Keep
+my email addresses private" is disabled in your
+`account settings <https://github.com/settings/emails>`_.
 
 .. _code review:
 



More information about the llvm-commits mailing list