fix(ci): authenticate CML release lookup - #981
Conversation
Git-Session-Id: 110d03e5-23eb-5721-b10f-c0dc3cb68927
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #981 +/- ##
=======================================
Coverage 53.11% 53.11%
=======================================
Files 48 48
Lines 2984 2984
Branches 681 681
=======================================
Hits 1585 1585
Misses 1379 1379
Partials 20 20 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Git-Session-Id: 110d03e5-23eb-5721-b10f-c0dc3cb68927
|
CI-green and mergeable — waiting only on a maintainer click. This PR is ready to merge, but the bot has pull-only access to this repo and can't self-merge — surfacing it here so it isn't lost. The monitoring loop will stop re-flagging it now that this note is posted. |
🤖 AI code reviewSafe to merge — no P0/P1 findingsConfidence 5/5 ✅ No findings. The diff looks correct to me on this pass. Reviewed Maintainer commands
|
|
@greptileai review |
Problem
The screenshot-comment workflow intermittently fails before downloading artifacts because
iterative/setup-cml@v3queries the public CML release API without authentication. The failed runs report an API rate-limit error for the shared runner IP, including run 29195897537 and run 34566780864.Fix
Pass the workflow's
GITHUB_TOKENtosetup-cml. The action already reads this environment variable when constructing Octokit, so release lookup gets the authenticated GitHub API quota.Verification
setup-cmlstep retainsvega: falseand now has the token environment variable.git diff --checkpasses.The original run is stale, so I did not rerun it; the next successful
Buildworkflow will exercise thisworkflow_runpath naturally.