Conversation
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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.
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. |
ThirdPartyNotices.txt(if applicable)Combines #1450 and #1414