ci: pin upload/download-artifact to v5.0.0 SHAs#262
Conversation
GitHub deprecated Node.js 20 actions on 2025-09-19. actions/upload-artifact@v4 and actions/download-artifact@v4 still ship Node 20 and trigger the deprecation warning on every job; the runner forces Node 24 from 2026-06-02. v5.0.0 of both actions is purely the Node 24 runtime bump (no artifact format change). Pinned to SHA to match the convention used elsewhere in this workflow (actions/checkout, actions/setup-python, actions/upload-pages-artifact, etc.). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 0/1 reviews remaining, refill in 44 minutes and 20 seconds.Comment |
☀️ Quality Summary 🚦 build-linux-gcc: 100% successful (✔️ 1025 passed, 🙈 3 skipped) Created by Quality Monitor v1.14.0 (#f3859fd). More details are shown in the GitHub Checks Result. |
☀️ Quality Summary 🚦 build-linux-gcc: 100% successful (✔️ 1025 passed, 🙈 3 skipped) Created by Quality Monitor v1.14.0 (#f3859fd). More details are shown in the GitHub Checks Result. |
* ci: add analyze-iwyu required check, bump cpputest-clang to sha-7eac3ab Adds the analyze-iwyu CI job (mirrors analyze-tidy/analyze-cppcheck shape) that runs `cmake --build --preset iwyu --target iwyu` against the new cpputest-clang image (which now ships include-what-you-use 0.23 built against clang-19). Job is wired into the summary's needs list so it gates the PR. The container SHA bump from sha-0385cea to sha-7eac3ab also updates the existing build-linux-clang job and the local clang dev-container. After this merges, branch protection should be updated to make analyze-iwyu a required status check (not done in this PR — it's a GitHub UI change, not a code change). * ci: surface analyze-iwyu in PR Quality Monitor summary CodeRabbit caught that analyze-iwyu was wired into the summary job's needs (gating execution) but the Quality Monitor summary itself neither downloaded the iwyu-report artifact nor had an analyze-iwyu analysis entry, so PR-level findings would be invisible. Adds the download step and the analysis block using warnings-ng's `iwyu` parser id. * ci: drop analyze-iwyu Quality Monitor analysis entry, keep artifact download The warnings-ng parser registry bundled with quality-monitor@v1 has no parser registered under id `iwyu` — the summary job was failing with `java.util.NoSuchElementException: No such parser registered: iwyu`. Dropping the analysis-array entry. The iwyu-report artifact download is kept for archival/inspection. The job-level pass/fail status of analyze-iwyu (which becomes a required check via branch protection) is the gate — Quality Monitor surfacing was nice-to-have for warnings-ng formatted output, not strictly necessary for required-check semantics. * ci: pin upload/download-artifact in analyze-iwyu and summary The analyze-iwyu job's iwyu-report upload and the summary job's iwyu-report download were still using @v4 floating tags. Pinning them to match the v5.0.0 SHAs introduced in #262 — keeps the action reference style uniform across the workflow.
1 participant
Summary
actions/upload-artifact@v4(×11) andactions/download-artifact@v4(×5) tov5.0.0, pinned by commit SHA to match the convention used forcheckout/setup-python/upload-pages-artifact.Reference: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
Test plan
yaml.safe_load)🤖 Generated with Claude Code