Repository navigation
ci: update the pinned actions and keep dependencies current with Dependabot - #141
Merged
Merged
Conversation
…ndabot actions/checkout v6.0.2 -> v7.0.1, actions/cache v5.0.4 -> v6.1.0, and actions/attest-build-provenance v4.1.1 -> v4.2.2, still SHA-pinned; neither major affects our triggers or inputs (checkout v7 only refuses fork code under pull_request_target/workflow_run, which we do not use; cache v6 is an ESM rework with an identical action.yml). setup-node v7.1.0 is current. .github/dependabot.yml proposes weekly version updates for npm (web), cargo, and github-actions, minor and patch grouped and majors separate, after a 7-day cooldown; the two web holds (TypeScript 7, @types/node above the Node 22 floor) are ignore rules with their reasons. CONTRIBUTING states the dependency policy. Refs #137 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This was referenced Oct 15, 2026
Closed
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Primary Issue
Refs #137
#137 stays open for one item: release attestation with
attest-build-provenancev4.2.2. Its steps run only on release tags (if: startsWith(github.ref, 'refs/tags/v')), so it is checked withgh attestation verifyat the next release, as on #95.Problem And Outcome
The pinned actions were behind. Dependabot was configured for security updates only, and the maintainer wants "latest versions of packages here where it makes sense in general". After merge:
dependabot.ymlwith their reasons;Changes
actions/checkout3d3c42e5…)actions/cache55cc8345…)actions/attest-build-provenance4d101475…)actions/setup-nodepull_request_targetorworkflow_run, unlessallow-unsafe-pr-checkoutis set. We use neither trigger. The runtime is still node24 andpersist-credentials: falseis kept.@actions/*majors. Itsaction.ymlis identical to v5.0.4.actions/attestpin moved. Its notes recommendactions/attestfor new work; that switch is not made here..github/dependabot.yml: weekly version updates for npm (/web), cargo (/), and github-actions (/).cooldownapplies.deps(web),deps(rust), andci.typescript >=7(TS 7 has no JS API, and svelte-check peers^5 || ^6) and@types/node >=23(it tracks the Node 22 floor).Scope
actions/attest, andrust-toolchain.tomlupdates. Dependabot does not manage the latter; deps(rust): move to the latest stable toolchain and current crates (sqlx 0.9 and five other majors) #136 set it to 1.99.Verification
683bc659a6b4802e6719e355cac5ca14933dfe92(tree3260cf5c).scripts/proof.pypassed 7/7 gates in 542 s.99156463…, archived on the maintainer machine.pr-gaterun exercises checkout v7, cache v6, and setup-node, andbuild-x86_64/build-aarch64exercise checkout v7.uses:matchesactions/<name>@<40-hex> # vX.Y.Z.Review And Merge Notes
action.yml. The integrator reviewed it and moved the policy line out of the enforcement list. No independent reviewer role for this CI-config change.🤖 Generated with Claude Code