ci: bump actions/upload-artifact from 4 to 7#51
Conversation
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 7. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v4...v7) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
📝 WalkthroughWalkthroughTwo GitHub Actions workflow files were updated to use Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/workflows/release.yml:
- Around line 68-71: The workflow currently uses mixed major versions of the
artifact actions (actions/upload-artifact@v7 vs actions/download-artifact@v4);
confirm whether this is intentional and either align them to the same major
release or add an explicit comment in the workflow explaining why the versions
differ; check the upload step (actions/upload-artifact@v7) and the corresponding
download step (actions/download-artifact@v4) and change one to match the other
or add a clarifying inline comment above these steps stating the reason for
intentionally using different majors.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 9dace794-7161-477e-b18f-527031c361de
📒 Files selected for processing (2)
.github/workflows/ci.yml.github/workflows/release.yml
| - uses: actions/upload-artifact@v7 | ||
| with: | ||
| name: ${{ matrix.name }} | ||
| path: ${{ matrix.name }}.tar.gz |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "== Artifact action versions in workflows =="
rg -n --iglob '*.yml' --iglob '*.yaml' 'actions/(upload|download)-artifact@' .github/workflows
echo
echo "== Any self-hosted runner usage in workflows =="
rg -n --iglob '*.yml' --iglob '*.yaml' 'self-hosted' .github/workflows || trueRepository: kienbui1995/mc-code
Length of output: 374
Clarify intentional mixed major versions for artifact actions.
Line 68 updates actions/upload-artifact@v7, while line 79 in the same workflow uses actions/download-artifact@v4. Confirm this mixed major version usage is intentional before merge. (No self-hosted runners are present in workflows, so runner version compatibility is not a concern.)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.github/workflows/release.yml around lines 68 - 71, The workflow currently
uses mixed major versions of the artifact actions (actions/upload-artifact@v7 vs
actions/download-artifact@v4); confirm whether this is intentional and either
align them to the same major release or add an explicit comment in the workflow
explaining why the versions differ; check the upload step
(actions/upload-artifact@v7) and the corresponding download step
(actions/download-artifact@v4) and change one to match the other or add a
clarifying inline comment above these steps stating the reason for intentionally
using different majors.
Bumps actions/upload-artifact from 4 to 7.
Release notes
Sourced from actions/upload-artifact's releases.
... (truncated)
Commits
043fb46Merge pull request #797 from actions/yacaovsnc/update-dependency634250cInclude changes in typespec/ts-http-runtime 0.3.5e454baaReadme: bump all the example versions to v7 (#796)74fad66Update the readme with direct upload details (#795)bbbca2dSupport direct file uploads (#764)589182cUpgrade the module to ESM and bump dependencies (#762)47309c9Merge pull request #754 from actions/Link-/add-proxy-integration-tests02a8460Add proxy integration testb7c566aMerge pull request #745 from actions/upload-artifact-v6-releasee516bc8docs: correct description of Node.js 24 support in READMEDependabot 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