Add optional TRIGGERED_BY_EMAIL to mention who triggered the build - #7
Open
AndreiHippo wants to merge 1 commit into
Open
Add optional TRIGGERED_BY_EMAIL to mention who triggered the build#7AndreiHippo wants to merge 1 commit into
AndreiHippo wants to merge 1 commit into
Conversation
Commits are not always made by a person. On GitHub, squash merges and merge commits made through the web UI are committed as `GitHub <noreply@github.com>`, and the author is whoever opened the pull request - which can itself be a bot. In that case neither of the two identities in the approval message points at the person waiting on the deploy, so nobody gets mentioned. Let CI pass that identity in via the new optional TRIGGERED_BY_EMAIL variable. When set, a `Triggered by` line is added to the message, mentioning the person if the email matches a Slack profile and falling back to the plain email if it does not - same behaviour as the existing committer and author lines. When it is unset, the message is unchanged. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The approval message identifies two people: the commit's committer and its author. Neither is reliably a person.
On GitHub, every commit created by the platform itself — squash merges, merge commits, rebase merges, web-editor edits, API-created commits — is committed as
GitHub <noreply@github.com>. On a protected branch that only ever receives merges through the merge button, that is every commit. Sampling ~200 commits onmainacross five of our repos, 185 hadnoreply@github.comas the committer; the handful that didn't were local pushes.The author is whoever opened the pull request, which is often a bot (Dependabot, a code-review app). So an approval request can arrive with no human mentioned at all, even though a person just pressed Merge and is waiting on the deploy:
That identity does exist — CI knows it (
github.triggering_actoron GitHub Actions,GITLAB_USER_EMAILon GitLab) — there's just no way to hand it to Magic Button.Change
A new optional
TRIGGERED_BY_EMAILvariable. When set, aTriggered byline is added to the details block, resolving the email to a Slack mention and falling back to the plain email — the same treatment the committer and author lines already get. When unset, the message is byte-for-byte what it is today.Testing
flake8andpylint -Eclean, perlint.sh.main.pywith Slack and git stubbed, across all three paths: email resolvable in Slack (mention), email unknown to Slack (plain-text fallback), and variable unset (unchanged output).