Skip to content

Fix refs value#3505

Merged
plengauer merged 3 commits into
mainfrom
plengauer-patch-3
May 26, 2026
Merged

Fix refs value#3505
plengauer merged 3 commits into
mainfrom
plengauer-patch-3

Conversation

@plengauer
Copy link
Copy Markdown
Owner

No description provided.

@plengauer plengauer self-assigned this May 25, 2026
@plengauer plengauer marked this pull request as ready for review May 25, 2026 09:14
@plengauer plengauer requested a review from moflwi as a code owner May 25, 2026 09:14
Copilot AI review requested due to automatic review settings May 25, 2026 09:14
@plengauer plengauer enabled auto-merge (squash) May 25, 2026 09:14
Copy link
Copy Markdown
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 PR adjusts GitHub Actions ref attribute formatting emitted by the workflow- and checksuite-level instrumentation to remove an extra leading /, bringing values closer to GitHub’s canonical refs/heads/<branch> format.

Changes:

  • Fix github.actions.workflow.ref construction to avoid @/refs/heads/... and use @refs/heads/....
  • Normalize several github.actions.*.ref attributes from /refs/heads/... to refs/heads/... in workflow and checksuite instrumentation.

Reviewed changes

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

File Description
actions/instrument/workflow/main.sh Removes the extra leading slash in workflow/event ref attributes.
actions/instrument/checksuite/main.sh Removes the extra leading slash in checksuite ref attributes.

Comment thread actions/instrument/workflow/main.sh
Comment thread actions/instrument/checksuite/main.sh
@plengauer plengauer merged commit 5442656 into main May 26, 2026
1137 of 1139 checks passed
@plengauer plengauer deleted the plengauer-patch-3 branch May 26, 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.

3 participants