chore(ci): bump actions/download-artifact from 4 to 8#49
chore(ci): bump actions/download-artifact from 4 to 8#49dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 4 to 8. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@v4...v8) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-version: '8' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
dependabot[bot] has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.
📝 WalkthroughWalkthroughThe GitHub Actions workflow ChangesGitHub Actions workflow upgrade
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.github/workflows/sovereign-pipeline.yml (1)
72-72: ⚡ Quick winConsider pinning the action to a commit SHA for security.
Static analysis flagged this unpinned action reference. Using version tags (v8) instead of commit SHAs makes the workflow vulnerable to tag poisoning attacks. While less critical for official GitHub actions, pinning to a SHA is a security best practice that ensures immutable, reproducible builds.
🔒 Proposed fix to pin to commit SHA
After verifying compatibility, update to use the commit SHA for v8.0.0:
- uses: actions/download-artifact@v8 + uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v8.0.0You can find the commit SHA for v8.0.0 by running:
gh api repos/actions/download-artifact/git/ref/tags/v8.0.0 --jq '.object.sha'As per coding guidelines, the static analysis tool flagged this as "required by blanket policy" for unpinned-uses.
🤖 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/sovereign-pipeline.yml at line 72, Replace the floating tag reference "uses: actions/download-artifact@v8" with an immutable commit SHA for the v8 release (e.g., actions/download-artifact@<commit-sha>) to prevent tag-poisoning; verify the chosen SHA corresponds to the intended v8.0.0 tag and is compatible with your workflow, then update the workflow to use that SHA instead of the v8 tag.
🤖 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/sovereign-pipeline.yml:
- Line 72: Replace the floating tag reference "uses:
actions/download-artifact@v8" with an immutable commit SHA for the v8 release
(e.g., actions/download-artifact@<commit-sha>) to prevent tag-poisoning; verify
the chosen SHA corresponds to the intended v8.0.0 tag and is compatible with
your workflow, then update the workflow to use that SHA instead of the v8 tag.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 56c0de65-94a8-4518-85fb-e1db0d23eb65
📒 Files selected for processing (1)
.github/workflows/sovereign-pipeline.yml
Bumps actions/download-artifact from 4 to 8.
Release notes
Sourced from actions/download-artifact's releases.
... (truncated)
Commits
3e5f45bAdd regression tests for CJK characters (#471)e6d03f6Add a regression test for artifact name + content-type mismatches (#472)70fc10cMerge pull request #461 from actions/danwkennedy/digest-mismatch-behaviorf258da9Add change docsccc058eFix linting issuesbd7976bAdd a setting to specify what to do on hash mismatch and default it toerrorac21fcfMerge pull request #460 from actions/danwkennedy/download-no-unzip15999bfAdd note about package bumps974686eBump the version tov8and add release notesfbe48b1Update test names to make it clearer what they doDependabot 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