Skip to content

Conversation

@avivkeller
Copy link
Member

workflow_id refers to the workflow, id refers to the run

Signed-off-by: Aviv Keller <me@aviv.sh>
Copilot AI review requested due to automatic review settings November 24, 2025 16:28
@avivkeller avivkeller requested a review from a team as a code owner November 24, 2025 16:28
@avivkeller avivkeller added the fast-track Fast Tracking PRs label Nov 24, 2025
@vercel
Copy link

vercel bot commented Nov 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Nov 24, 2025 4:32pm

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 PR fixes a bug in the bundle-compare workflow where the incorrect GitHub Actions context property was used to reference a workflow run. The change corrects workflow_run.workflow_id to workflow_run.id for the run-id parameter.

  • Corrects the run-id parameter to use the workflow run identifier instead of the workflow template identifier

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Nov 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.26%. Comparing base (8504a85) to head (5ad34f4).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8365      +/-   ##
==========================================
- Coverage   76.29%   76.26%   -0.03%     
==========================================
  Files         118      118              
  Lines        9903     9903              
  Branches      336      336              
==========================================
- Hits         7555     7553       -2     
- Misses       2346     2348       +2     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Aviv Keller <me@aviv.sh>
@avivkeller avivkeller enabled auto-merge November 24, 2025 16:31
@avivkeller avivkeller added this pull request to the merge queue Nov 24, 2025
Merged via the queue into main with commit 76ac237 Nov 24, 2025
12 checks passed
@avivkeller avivkeller deleted the fix/workflow_id/run_id branch November 24, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants