Upgrade actions/checkout to v6 across all GitHub workflows#2168
Upgrade actions/checkout to v6 across all GitHub workflows#2168elazarlachkar merged 1 commit intomasterfrom
Conversation
Addresses the Node.js 20 deprecation warning by upgrading to actions/checkout@v6 which runs on Node.js 24. Updated workflows: - release.yml (4 usages) - test-warehouse.yml (2 usages) - test-github-action.yml (2 usages) - bump-version.yml (2 usages) - build-docker-image.yml (1 usage) - cleanup-stale-schemas.yml (1 usage) - run-precommit.yml (1 usage) - update_pylon_issue.yml (1 usage, was v2) Co-Authored-By: Elazar Lachkar <elazar098@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
👋 @devin-ai-integration[bot] |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughEight GitHub Actions workflow files were updated to use Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
Upgrades
actions/checkoutto v6 in all GitHub workflow files to resolve the Node.js 20 deprecation warning:actions/checkout@v6runs on Node.js 24. Updated 14 usages across 8 workflow files:release.ymltest-warehouse.ymltest-github-action.ymlbump-version.ymlbuild-docker-image.ymlcleanup-stale-schemas.ymlrun-precommit.ymlupdate_pylon_issue.ymlThe only breaking change in v6 (changelog) is that credentials are now persisted to a separate file instead of the main
.gitconfig. This should be transparent for all workflows here.Review & Testing Checklist for Human
bump-version.ymlstill works — it doesgit pushafter checkout, which relies on persisted credentials. The v6 credential storage change (separate file) should be compatible, but worth confirming on next version bump.run-precommitCI job passes on this PR (it's the only workflow triggered bypull_request)Notes
update_pylon_issue.ymlwas still on v2 — jumped directly to v6.release,test-warehouse,bump-version, etc.) are only triggered manually or by tags/schedules, so they won't be exercised by this PR's CI.Resolves CORE-529
Link to Devin session: https://app.devin.ai/sessions/9dfddb668bfd458498b56306dd4e0026
Requested by: @elazarlachkar
Summary by CodeRabbit