Skip to content

Bump CI action dependencies to latest major versions#4531

Draft
tnull wants to merge 1 commit intolightningdevkit:mainfrom
tnull:2026-04-bump-ci-workflows
Draft

Bump CI action dependencies to latest major versions#4531
tnull wants to merge 1 commit intolightningdevkit:mainfrom
tnull:2026-04-bump-ci-workflows

Conversation

@tnull
Copy link
Copy Markdown
Contributor

@tnull tnull commented Apr 1, 2026

GitHub is deprecating Node.js 20 for Actions runners, with enforcement starting June 2026. Bump all action dependencies to their latest major versions which use Node.js 22+.

Co-Authored-By: HAL 9000

GitHub is deprecating Node.js 20 for Actions runners, with enforcement
starting June 2026. Bump all action dependencies to their latest major
versions which use Node.js 22+.

Co-Authored-By: HAL 9000
@tnull tnull requested a review from joostjager April 1, 2026 08:51
@ldk-reviews-bot
Copy link
Copy Markdown

ldk-reviews-bot commented Apr 1, 2026

👋 Hi! This PR is now in draft status.
I'll wait to assign reviewers until you mark it as ready for review.
Just convert it out of draft status when you're ready for review!

cargo clean
- name: Download honggfuzz corpus
uses: actions/download-artifact@v4
uses: actions/download-artifact@v8
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The fuzz job uploads with actions/upload-artifact@v7 (line 261) while the coverage job downloads with actions/download-artifact@v8 (line 86). Historically these two actions have been versioned in lockstep (v1/v1, v2/v2, v3/v3, v4/v4). The major version mismatch is unusual — please verify that upload-artifact v7 and download-artifact v8 are artifact-format compatible, or confirm whether these actions have intentionally diverged in versioning.

- uses: actions/checkout@v4
- uses: rustsec/audit-check@v1.4.1
- uses: actions/checkout@v6
- uses: rustsec/audit-check@v2
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Major version bump to rustsec/audit-check@v2 — please verify that the ignore parameter syntax is still supported in v2. If the v2 API changed (e.g., switching to a different input name or format for advisory ignores), this could silently stop ignoring RUSTSEC-2021-0145 and cause spurious CI failures.

Also note: the comment on lines 22-23 still references MSRV rustc 1.63, but the project MSRV elsewhere in build.yml is 1.75. This is a pre-existing issue, not introduced by this PR.

@ldk-claude-review-bot
Copy link
Copy Markdown
Collaborator

Review Summary

Two inline comments posted:

  1. .github/workflows/build.yml:86upload-artifact@v7 / download-artifact@v8 version mismatch. These actions have historically been versioned in lockstep. The fuzz job uploads at v7 and the coverage job downloads at v8 — verify these are artifact-format compatible.

  2. .github/workflows/audit.yml:15rustsec/audit-check@v2 is a major version bump. Verify the ignore parameter is still supported with the same syntax in v2.

Cross-cutting concern

Version existence should be verified: Several of these version bumps are aggressive jumps from v4 (actions/checkout@v6 is +2 major versions, actions/upload-artifact@v7 is +3, actions/download-artifact@v8 is +4). Please confirm all target versions actually exist and are released. If any version tag doesn't exist, the workflow will fail silently at the checkout step of the action.

@tnull
Copy link
Copy Markdown
Contributor Author

tnull commented Apr 1, 2026

Need to double-check the bumped versions. Could be Claude messed up.

@tnull tnull marked this pull request as draft April 1, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants