Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Dec 15, 2025

Bumps JamesIves/github-pages-deploy-action from 4.7.4 to 4.7.6.

Release notes

Sourced from JamesIves/github-pages-deploy-action's releases.

v4.7.6

What's Changed

Build 🔧

Full Changelog: JamesIves/github-pages-deploy-action@v4...v4.7.6

v4.7.5

What's Changed

Bug Fixes 🐛

Build 🔧

Full Changelog: JamesIves/github-pages-deploy-action@v4...v4.7.5

Commits
  • 9d877ee Deploy Production Code for Commit 5ad124cd797fc1553b9810f7cb262a32d1432d1d 🚀
  • 5ad124c Merge branch 'dev' into releases/v4
  • 83e989f fix: additional fixes for stripping git configs
  • 286f4ba build(deps): bump typescript-eslint in the typescript group (#1930)
  • 4ef313c Deploy Production Code for Commit f58e95ea79869a93f918a2f71af6035e58be1fae 🚀
  • f58e95e Merge branch 'dev' into releases/v4
  • f33629c build(deps): bump actions/setup-node from 6.0.0 to 6.1.0 (#1927)
  • 771d2b1 build(deps): bump typescript-eslint in the typescript group (#1925)
  • 1d137fd fix: cross-repo deployment with actions/checkout@v6 includeIf credentials (#1...
  • f43d9b3 build(deps): bump actions/checkout from 6.0.0 to 6.0.1 (#1926)
  • Additional commits viewable in compare view

Dependabot compatibility score

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 rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will 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 version will 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 dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [JamesIves/github-pages-deploy-action](https://github.com/jamesives/github-pages-deploy-action) from 4.7.4 to 4.7.6.
- [Release notes](https://github.com/jamesives/github-pages-deploy-action/releases)
- [Commits](JamesIves/github-pages-deploy-action@v4.7.4...v4.7.6)

---
updated-dependencies:
- dependency-name: JamesIves/github-pages-deploy-action
  dependency-version: 4.7.6
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Dec 15, 2025

- name: Deploy to GitHub pages 🚀
uses: JamesIves/[email protected].4
uses: JamesIves/[email protected].6
Copy link
Member

@jgabry jgabry Dec 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@VisruthSK Is it safe to merge these updates to the deploy-action in general? Also is there a reason why you have this action set to a specific minor version, whereas e.g. for loo it's just @v4 instead of @4.7.4? Just curious if that's intentional.

Copy link
Member

@VisruthSK VisruthSK Dec 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not intentional and doesn't provide much (any?) security against release-jacking so I see no reason to. I may have to think about that more, but maybe it does actually sorta protect against release-jacking if the compromised release is a minor or patch version the next build would silently move to it, whereas here we'd get a PR and have to approve manually. Maybe we should do this everywhere? Adds some friction/regular maintanance though.

I probably copied it from the wrong place.

We could run the pkgdown action off this branch to see if the site builds and gets deployed before merging? Since its just a patch I doubt anything would break but can't hurt.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually yeah maybe it's good to do it this way not just for bayesplot but for all of them for the reason you mentioned.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nevermind, I forgot that release aren't immutable so nothing matters--in a scenario where the attacker has repo access, they could just change the files in the latest release instead of making a new one. So maybe just move to major version tags to make it smoother and skip these PRs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants