-
-
Notifications
You must be signed in to change notification settings - Fork 53
⬆️ Bump actions/upload-artifact from 4 to 5 #659
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
⬆️ Bump actions/upload-artifact from 4 to 5 #659
Conversation
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v4...v5) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Dependabot PRs cannot access repository secrets due to GitHub's security model. This causes the Netlify preview deployment to fail with 'Unauthorized' errors. Changes: - Add conditional check to skip Netlify deploy when actor is dependabot[bot] - Also skip for PRs from forks or when secrets are unavailable - Add informative skip message step for transparency - Only post PR comment when deployment actually succeeds - Move secrets to job-level env for proper conditional access This ensures: - Dependabot PRs pass CI without Netlify preview (acceptable trade-off) - Regular contributor PRs continue to get Netlify previews - No security risks from exposing secrets to untrusted actors - Clear logging when deployment is skipped Fixes the issue reported in PR #659
Dependabot PRs cannot access repository secrets due to GitHub's security model. This causes the Netlify preview deployment to fail with 'Unauthorized' errors. Changes: - Add conditional check to skip Netlify deploy when actor is dependabot[bot] - Also skip for PRs from forks or when secrets are unavailable - Add informative skip message step for transparency - Only post PR comment when deployment actually succeeds - Move secrets to job-level env for proper conditional access This ensures: - Dependabot PRs pass CI without Netlify preview (acceptable trade-off) - Regular contributor PRs continue to get Netlify previews - No security risks from exposing secrets to untrusted actors - Clear logging when deployment is skipped Fixes the issue reported in PR #659
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
|
📖 Netlify Preview Ready! Preview URL: https://pr-659--sunny-cactus-210e3e.netlify.app (f748c67) |
|
📖 Netlify Preview Ready! Preview URL: https://pr-659--sunny-cactus-210e3e.netlify.app (c381d90) |
Bumps actions/upload-artifact from 4 to 5.
Release notes
Sourced from actions/upload-artifact's releases.
... (truncated)
Commits
330a01cMerge pull request #734 from actions/danwkennedy/prepare-5.0.003f2824Updategithub.dep.yml905a1ecPreparev5.0.02d9f9cdMerge pull request #725 from patrikpolyak/patch-19687587Merge branch 'main' into patch-12848b2cMerge pull request #727 from danwkennedy/patch-19b51177Spell out the first use of GHEScd231caUpdate GHES guidance to include reference to Node 20 versionde65e23Merge pull request #712 from actions/nebuk89-patch-18747d8cUpdate README.mdDependabot 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 mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@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)