ci: bump actions/upload-pages-artifact from 3 to 5#76
ci: bump actions/upload-pages-artifact from 3 to 5#76dependabot[bot] wants to merge 1 commit intomainfrom
Conversation
Bumps [actions/upload-pages-artifact](https://github.com/actions/upload-pages-artifact) from 3 to 5. - [Release notes](https://github.com/actions/upload-pages-artifact/releases) - [Commits](actions/upload-pages-artifact@v3...v5) --- updated-dependencies: - dependency-name: actions/upload-pages-artifact dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
📝 WalkthroughWalkthroughUpdated the GitHub Actions workflow to use Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 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 (1)
.github/workflows/pages.yml (1)
27-29: Consider explicitly settinginclude-hidden-filesfor clarity, though not critical here.
actions/upload-pages-artifact@v5changed hidden-file handling in v4+. The workflow copiesdocs/sitedirectly to_site, and a scan confirmsdocs/sitecontains no hidden files, so there is no immediate risk. Settinginclude-hidden-files: false(ortrueif needed in future) clarifies intent and protects against accidental changes if hidden files are added todocs/sitelater.Optional: Add explicit intent
- uses: actions/upload-pages-artifact@v5 with: path: _site include-hidden-files: false🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/workflows/pages.yml around lines 27 - 29, The upload step using actions/upload-pages-artifact@v5 currently specifies only "with: path: _site"; explicitly add the include-hidden-files input to that step (e.g., include-hidden-files: false) so the workflow documents the intended hidden-file behavior and prevents accidental inclusion of dotfiles if docs/site changes later; update the same step that references actions/upload-pages-artifact@v5 and the existing "with: path: _site" block to include the new include-hidden-files key.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.github/workflows/pages.yml:
- Around line 27-29: The upload step using actions/upload-pages-artifact@v5
currently specifies only "with: path: _site"; explicitly add the
include-hidden-files input to that step (e.g., include-hidden-files: false) so
the workflow documents the intended hidden-file behavior and prevents accidental
inclusion of dotfiles if docs/site changes later; update the same step that
references actions/upload-pages-artifact@v5 and the existing "with: path: _site"
block to include the new include-hidden-files key.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 36975c3b-63bf-4e1a-abbe-15c70914a4e3
📒 Files selected for processing (1)
.github/workflows/pages.yml
|
Superseded by #78. Same root cause as #71/#75: Dependabot-triggered workflows don't receive Generated by Claude Code |
|
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. |
Supersedes #76 (same Dependabot SONAR_TOKEN issue as #71/#75). The v4 release excludes hidden files (dotfiles) by default. The workflow uploads docs/site/, which contains only index.html — no dotfiles — so the breaking change does not affect this pipeline. https://claude.ai/code/session_01R2n6wKqFkYPvHkwaip8EnJ
…) (#78) * chore(deps): bump actions/upload-pages-artifact from v3 to v5 Supersedes #76 (same Dependabot SONAR_TOKEN issue as #71/#75). The v4 release excludes hidden files (dotfiles) by default. The workflow uploads docs/site/, which contains only index.html — no dotfiles — so the breaking change does not affect this pipeline. https://claude.ai/code/session_01R2n6wKqFkYPvHkwaip8EnJ * chore(deps): bump rustls-webpki to 0.103.13 for RUSTSEC-2026-0104 Fresh transitive advisory: panic during CRL parsing. Blocks License & Supply Chain and Dependency Audit on main, not specific to this PR. Folding the fix here to unblock CI. https://claude.ai/code/session_01R2n6wKqFkYPvHkwaip8EnJ --------- Co-authored-by: Claude <noreply@anthropic.com>
Bumps actions/upload-pages-artifact from 3 to 5.
Release notes
Sourced from actions/upload-pages-artifact's releases.
Commits
fc324d3Merge pull request #139 from Tom-van-Woudenberg/patch-1fe9d4b7Merge branch 'main' into patch-10ca1617Merge pull request #137 from jonchurch/include-hidden-files57f0e84Update action.yml4a90348v7 --> hash56f665aUpdate upload-artifact action to version 7f7615f5Addinclude-hidden-filesinput7b1f4a7Merge pull request #127 from heavymachinery/pin-sha4cc19c7Pinactions/upload-artifactto SHA2d163beMerge pull request #107 from KittyChiu/mainDependabot 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