-
Notifications
You must be signed in to change notification settings - Fork 123
Bump actions/checkout from 4 to 5 #1463
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this 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 pull request upgrades GitHub Actions versions across the CI/CD workflow files. The changes update actions/checkout from v4 to v5, actions/setup-dotnet from v4 to v5, and stefanzweifel/git-auto-commit-action from v5 to v7. Additionally, it refines conditional logic in the CI workflow to properly scope test coverage and result uploads to only the C# build matrix entries, and removes persist-credentials: false from the publish job's checkout step to enable automated commits using a custom token.
- Updated core GitHub Actions (
actions/checkout,actions/setup-dotnet) to their latest major versions - Added
matrix.language == 'csharp'condition to test coverage and upload steps to avoid redundant operations for non-C# matrix entries - Removed
persist-credentials: falsein the publish job to allow git-auto-commit-action to function correctly
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/integration-tests.yml | Updated actions/checkout from v4 to v5 in two locations |
| .github/workflows/copilot-setup-steps.yml | Updated actions/checkout from v4 to v5 |
| .github/workflows/CI.yml | Updated actions/checkout (v4→v5), actions/setup-dotnet (v4→v5), git-auto-commit-action (v5→v7); added matrix.language == 'csharp' conditions to test-related steps; removed persist-credentials: false from publish job checkout |
| - uses: actions/checkout@v5 | ||
|
|
||
| - name: Setup .NET | ||
| uses: actions/setup-dotnet@v2 |
Copilot
AI
Dec 3, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent action version: actions/setup-dotnet@v2 should be updated to @v5 to match the upgrades in CI.yml (lines 38, 111, 158, 306). This ensures consistency across all workflow files and provides the latest features and fixes.
| uses: actions/setup-dotnet@v2 | |
| uses: actions/setup-dotnet@v5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will update in separate dependabot #1175
Unit Test Results 1 files 1 suites 10m 24s ⏱️ Results for commit 7070eff. ♻️ This comment has been updated with latest results. |
xiaonile
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
ThirdPartyNotices.txt(if applicable)Combines #1450 and #1414