Skip to content

Conversation

@petetomasik
Copy link
Contributor

Fixes #70

GitHub events like status don't include a pusher field in their payload, causing the action to send empty author information to the Buildkite API. This results in builds being attributed to the API token owner instead of the actual commit author.

Changes

  • Added two new optional input parameters: commit_author_name and commit_author_email
  • Refactored author extraction into separate get_author_name() and get_author_email() functions
  • Added tests
  • Updated documentation to explain new processing order

New processing order

  1. .pusher.name and .pusher.email (existing behavior)
  2. .head_commit.author.name and .head_commit.author.email (for push events)
  3. .commit.commit.author.name and .commit.commit.author.email (for status events)
  4. commit_author_name and commit_author_email input parameters (user-provided)
  5. Git commit information from the repository

NOTE: Both name and email are processed as separate values. This means that if only one of commit_author_name or commit_author_email is provided as an input parameter the other value could resolve to a different "author".

@petetomasik petetomasik requested a review from a team as a code owner October 24, 2025 15:57
Copy link

@jeremybumsted jeremybumsted left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice one!

@petetomasik petetomasik merged commit f0dcd5a into master Oct 24, 2025
2 checks passed
@petetomasik petetomasik deleted the SUP-5035-author-name-email-defaults branch October 24, 2025 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

github status event does not have "pusher" field, so author fields are empty and all builds have token owner username

2 participants