Skip to content

feat(github): Add event tag to Sentry payload#75

Merged
kenzoengineer merged 1 commit intomainfrom
kjiang/add_event_tag
Mar 20, 2026
Merged

feat(github): Add event tag to Sentry payload#75
kenzoengineer merged 1 commit intomainfrom
kjiang/add_event_tag

Conversation

@kenzoengineer
Copy link
Member

It is immediately useful to me to know how an action was triggered, whether it was a manual dispatch, on a PR, or push to the branch.

This adds the supported event tag, and a test to verify functionality.

@kenzoengineer kenzoengineer requested a review from a team as a code owner March 19, 2026 21:49
@kenzoengineer kenzoengineer requested a review from armenzg March 19, 2026 21:49
"commit": runs["head_sha"],
"repo": repo,
"run_attempt": runs["run_attempt"], # Rerunning a job
"event": runs["event"],
Copy link
Member

Choose a reason for hiding this comment

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

Do you want a more self-explanatory value? Using event as a tag may not be clear.

Copy link
Member Author

Choose a reason for hiding this comment

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

ahh sorry merged before reading this comment, i want to keep it the same because its the nomenclature github uses across it's platform:
image

"source" or "type" might be more ambiguous and confusing

Copy link
Member

@armenzg armenzg left a comment

Choose a reason for hiding this comment

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

Feel free to merge and deploy

@kenzoengineer kenzoengineer merged commit 6258840 into main Mar 20, 2026
12 checks passed
@kenzoengineer kenzoengineer deleted the kjiang/add_event_tag branch March 20, 2026 17:38
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.

2 participants