Skip to content

Update GitHub Actions versions #2240

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PixelPil0t1
Copy link

Update GitHub Actions versions

@landabaso
Copy link
Member

Hey @junderw this looks good to me. It fixes the red CI runs (e.g. #2248 , #2247 ). I applied the same changes in my fork and the workflow went green: https://github.com/landabaso/bitcoinjs-lib/actions/runs/16500766519

main_ci.yml currently pins v2/v3 Actions to specific SHAs. Was there a particular reason (security/reproducibility)? If not, moving to the v4 tags (or pinning the latest v4 SHAs) seems fine. Once this merges, the failing workflows on master should pass again.

@junderw
Copy link
Member

junderw commented Jul 26, 2025

LGTM if we want to trust the repo maintainers of those actions.

Some users complained and I saw nothing wrong with pinning, so I allowed it.

This will probably upset those users.

I am indifferent.

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