Skip to content

Update & Pin GHA deps - #1492

Merged
avivajpeyi merged 2 commits into
TeamCOMPAS:devfrom
ulgens:update-gha-deps
Oct 27, 2026
Merged

avivajpeyi merged 2 commits into
TeamCOMPAS:devfrom
ulgens:update-gha-deps

Conversation

@ulgens

@ulgens ulgens commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Resolves deprecation warnings, improves supply chain security and an overall cleaning & update operation.

Can be automated via renovate: https://docs.renovatebot.com/modules/manager/github-actions/

Fixes https://docs.zizmor.sh/audits/#unpinned-uses and increases supply chain security
Resolves

> Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24...

warnings and updates the dependencies.

This operation can be automated via renovate: https://docs.renovatebot.com/modules/manager/github-actions/
@ulgens
ulgens marked this pull request as ready for review September 24, 2026 13:54

@ilyamandel ilyamandel left a comment

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.

Sorry for the long silence, @ulgens , I was secretly hoping that someone with more GitHub workflow knowledge than me (@avivajpeyi ? @reinhold-willcox ?) might review these; but happy to trust you!

@warrickball

Copy link
Copy Markdown
Collaborator

At first I was a bit surprised by the pinning of particular commit hashes rather than version tags but I've read around and it appears this is becoming standard practice, even though I'd actually never seen it before. All the major projects I had a quick look at (e.g. NumPy, requests, LLVM) use this idiom and it appears that dependabot supports it.

I've checked that the hashes correspond to the listed versions, so I personally think this is good to merge but I'm also not a core dev.

@avivajpeyi
avivajpeyi merged commit c355e20 into TeamCOMPAS:dev Oct 27, 2026
4 checks passed
@ulgens
ulgens deleted the update-gha-deps branch October 27, 2026 04:36
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.

4 participants