chore(deps): bump actions/checkout from 6 to 7#21
Conversation
Bumps [actions/checkout](https://github.com/actions/checkout) from 6 to 7. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v6...v7) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
📝 WalkthroughWalkthroughThree checkout steps across two GitHub Actions workflow files ( Changesactions/checkout v7 upgrade
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (2)
.github/workflows/static.yml (1)
14-14: 🧹 Nitpick | 🔵 Trivial | ⚡ Quick winConsider adding
persist-credentials: falsefor credential security.Both checkout steps should explicitly disable credential persistence to prevent Git tokens from being accessible to subsequent workflow steps. This is a security best practice, especially in workflows that may execute untrusted code or have multiple steps.
🔐 Proposed fix for the phpstan job (line 14)
- uses: actions/checkout@v7 + with: + persist-credentials: false🔐 Proposed fix for the ecs job (line 35)
- name: Checkout uses: actions/checkout@v7 + with: + persist-credentials: falseAlso applies to: 35-35
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/workflows/static.yml at line 14, Both instances of the actions/checkout@v7 step in the workflow (in the phpstan job at line 14 and in the ecs job at line 35) need to include the persist-credentials: false parameter to prevent Git tokens from being exposed to subsequent workflow steps. Add this configuration parameter to both checkout steps to follow security best practices and ensure credentials are not persisted across the workflow execution..github/workflows/tests.yml (1)
22-22: 🧹 Nitpick | 🔵 Trivial | ⚡ Quick winConsider adding
persist-credentials: falsefor credential security.The checkout step should explicitly disable credential persistence to prevent Git tokens from being accessible to subsequent workflow steps. This is a security best practice, especially in test workflows.
🔐 Proposed fix
- name: Checkout code uses: actions/checkout@v7 + with: + persist-credentials: false🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/workflows/tests.yml at line 22, The actions/checkout@v7 step in the tests.yml workflow is missing a security configuration that prevents Git credentials from being persisted in the workflow environment. Add the `persist-credentials: false` option to the checkout step to ensure that Git tokens are not accessible to subsequent workflow steps. This should be added as a `with:` clause under the `uses: actions/checkout@v7` line.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.github/workflows/static.yml:
- Line 14: Both instances of the actions/checkout@v7 step in the workflow (in
the phpstan job at line 14 and in the ecs job at line 35) need to include the
persist-credentials: false parameter to prevent Git tokens from being exposed to
subsequent workflow steps. Add this configuration parameter to both checkout
steps to follow security best practices and ensure credentials are not persisted
across the workflow execution.
In @.github/workflows/tests.yml:
- Line 22: The actions/checkout@v7 step in the tests.yml workflow is missing a
security configuration that prevents Git credentials from being persisted in the
workflow environment. Add the `persist-credentials: false` option to the
checkout step to ensure that Git tokens are not accessible to subsequent
workflow steps. This should be added as a `with:` clause under the `uses:
actions/checkout@v7` line.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 26586a78-169f-42c0-8151-8e85dc2bdd86
📒 Files selected for processing (2)
.github/workflows/static.yml.github/workflows/tests.yml
Bumps actions/checkout from 6 to 7.
Release notes
Sourced from actions/checkout's releases.
Changelog
Sourced from actions/checkout's changelog.
... (truncated)
Commits
9c091bbupdate error wording (#2467)1044a6dgetting ready for checkout v7 release (#2464)f028218Bump the minor-npm-dependencies group across 1 directory with 3 updates (#2462)d914b26upgrade module to esm and update dependencies (#2463)537c7efBump@actions/coreand@actions/tool-cacheand Remove uuid (#2459)130a169Bump js-yaml from 4.1.0 to 4.2.0 (#2461)7d09575Bump flatted from 3.3.1 to 3.4.2 (#2460)0f9f3aaBump actions/publish-immutable-action (#2458)f9e715ablock checking out fork pr for pull_request_target and workflow_run (#2454)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Summary by CodeRabbit