Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Report job level annotations#4216

Merged
TingluoHuang merged 1 commit intomainfrom
users/tihuang/setuptelemtry
Jan 27, 2026
Merged

Report job level annotations#4216
TingluoHuang merged 1 commit intomainfrom
users/tihuang/setuptelemtry

Conversation

@TingluoHuang
Copy link
Member

image

instead of

image

@TingluoHuang TingluoHuang requested a review from a team as a code owner January 26, 2026 22:39
Copilot AI review requested due to automatic review settings January 26, 2026 22:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds support for reporting job-level annotations in addition to the existing task-level annotations. The change is gated behind a new feature flag to allow gradual rollout.

Changes:

  • Added a new feature flag SendJobLevelAnnotations to control the behavior
  • Modified string literal comparison to use a constant for better maintainability
  • Implemented job-level annotation collection that converts issues to annotations when the feature flag is enabled

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/Runner.Common/Constants.cs Added new feature flag SendJobLevelAnnotations to the Features class
src/Runner.Worker/ExecutionContext.cs Replaced string literal with constant, added job-level annotation processing logic gated by the new feature flag

@TingluoHuang TingluoHuang merged commit 6822f4a into main Jan 27, 2026
18 checks passed
@TingluoHuang TingluoHuang deleted the users/tihuang/setuptelemtry branch January 27, 2026 21:52
dawidmalina pushed a commit to dawidmalina/github-runner that referenced this pull request Feb 26, 2026
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.

3 participants