Skip to content

Attribute release-bump commit to the gh-authenticated user - #3500

Merged
ehl-jf merged 1 commit into
masterfrom
fix-cla-on-releases
May 21, 2026
Merged

ehl-jf merged 1 commit into
masterfrom
fix-cla-on-releases

Conversation

@ehl-jf

@ehl-jf ehl-jf commented May 20, 2026

Copy link
Copy Markdown
Contributor

The release-prep script hardcoded the git committer to github-actions[bot] but used the running gh token to open the PR and add the "ignore for release" label. Locally that produced PRs where the commit, the PR opener, and the label event disagreed.

Resolve the gh identity once via gh api user, then set GIT_AUTHOR_/GIT_COMMITTER_ env vars on the bump commit. In CI with GITHUB_TOKEN the user is github-actions[bot] (unchanged); locally it's the real user. Fall back to github-actions[bot] if gh api user fails.

  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • The pull request is targeting the master branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....

The release-prep script hardcoded the git committer to
github-actions[bot] but used the running gh token to open the PR and
add the "ignore for release" label. Locally that produced PRs where
the commit, the PR opener, and the label event disagreed.

Resolve the gh identity once via `gh api user`, then set
GIT_AUTHOR_*/GIT_COMMITTER_* env vars on the bump commit. In CI with
GITHUB_TOKEN the user is github-actions[bot] (unchanged); locally it's
the real user. Fall back to github-actions[bot] if `gh api user` fails.
@ehl-jf ehl-jf added ignore for release Automatically generated release notes safe to test Approve running integration tests on a pull request labels May 20, 2026
@ehl-jf
ehl-jf merged commit ebc4ac7 into master May 21, 2026
111 of 113 checks passed
@ehl-jf
ehl-jf deleted the fix-cla-on-releases branch May 21, 2026 09:43

This branch had an error being deployed

1 failed deployment
frogbot — dc9bb729 Deployed May 20, 2026 by ehl-jf via scan-pull-request #4275
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore for release Automatically generated release notes safe to test Approve running integration tests on a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant