Skip to content

ci: update GitHub Actions versions - #174

Merged
benthecarman merged 1 commit into
lightningdevkit:mainfrom
vincenzopalazzo:codex/update-actions-155
Mar 26, 2026
Merged

ci: update GitHub Actions versions#174
benthecarman merged 1 commit into
lightningdevkit:mainfrom
vincenzopalazzo:codex/update-actions-155

Conversation

@vincenzopalazzo

Copy link
Copy Markdown
Member

Summary

  • update all workflows to use the latest major of actions/checkout
  • update actions/cache and peter-evans/create-pull-request where newer majors are available
  • set explicit read-only contents permissions on non-mutating workflows

Testing

  • go run github.com/rhysd/actionlint/cmd/actionlint@latest
  • git diff --check

Closes #155

@ldk-reviews-bot

ldk-reviews-bot commented Mar 25, 2026

Copy link
Copy Markdown

I've assigned @benthecarman as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

@benthecarman

Copy link
Copy Markdown
Collaborator

Can we also add so we don't get duplicate runs for multiple pushes

concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true

@vincenzopalazzo

vincenzopalazzo commented Mar 26, 2026

Copy link
Copy Markdown
Member Author

Fixed in 93ba2d3. Added the same concurrency cancellation block to build.yml and check-protos.yml so duplicate runs from successive pushes get canceled. I left cron-weekly-rustfmt.yml unchanged since it only runs on schedule/workflow_dispatch.

@vincenzopalazzo
vincenzopalazzo force-pushed the codex/update-actions-155 branch from 4e0a332 to 93ba2d3 Compare March 26, 2026 14:27
@benthecarman
benthecarman merged commit 2a5405f into lightningdevkit:main Mar 26, 2026
7 checks passed
@vincenzopalazzo
vincenzopalazzo deleted the codex/update-actions-155 branch March 27, 2026 13:13
rsafier pushed a commit to rsafier/ldk-server that referenced this pull request Apr 20, 2026
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.

Update actions steps to latest versions

3 participants