Skip to content

ci: update actions to Node 24 releases pinned by SHA - #15

Merged
ChiragAgg5k merged 1 commit into
mainfrom
ci/update-action-pins
Nov 6, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
ci/update-action-pins

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

The release build warns that its actions target Node 20, which is deprecated. This moves every action in the repo to its latest release and pins it by SHA, with the version in a comment.

build-and-push.yml only runs on a published release, so this PR does not exercise it. It takes effect on the next release.

The old tags target Node 20, which GitHub now forces onto Node 24 with a deprecation warning.
@appwrite

appwrite Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The updated action references resolve to the stated releases, and the workflow inputs remain compatible with these versions.

Updated the GitHub Actions workflows to use Node 24-compatible action releases pinned to commit SHAs, with release versions noted in comments. The release publishing workflow and the checkout steps used by lint, static analysis, and E2E workflows are covered.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 4
File Change
.github/workflows/build-and-push.yml Pinned checkout and Docker publishing actions to their Node 24 releases.
.github/workflows/codeql.yml Pinned the checkout action to v7.0.1.
.github/workflows/linter.yml Pinned the checkout action to v7.0.1.
.github/workflows/tests.yml Pinned the checkout action to v7.0.1.

Reviewed 2a18a17 · Details · Comment @appwrite review to re-run, or mention @appwrite with a question.

@appwrite appwrite Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 3d27075 into main Nov 6, 2026
4 checks passed
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.

1 participant