Skip to content

Use Node 24 reusable package workflow#43

Merged
Jess Sullivan (Jesssullivan) merged 1 commit into
mainfrom
jess/node24-ci-template
May 1, 2026
Merged

Use Node 24 reusable package workflow#43
Jess Sullivan (Jesssullivan) merged 1 commit into
mainfrom
jess/node24-ci-template

Conversation

@Jesssullivan

Copy link
Copy Markdown
Contributor

Summary

Bumps the reusable js-bazel-package workflow pin to the ci-templates merge commit that upgrades JavaScript actions to Node 24 runtimes.

  • updates .github/workflows/ci.yml
  • updates .github/workflows/publish.yml
  • keeps both workflows pinned to the same shared-template SHA

Upstream template change: tinyland-inc/ci-templates#27.

Validation

  • pnpm run check:release-metadata
  • ruby -e 'require "yaml"; %w[.github/workflows/ci.yml .github/workflows/publish.yml].each { |path| YAML.load_file(path); puts "#{path}: ok" }'
  • git diff --check

Notes

This should remove the Node.js 20 JavaScript action deprecation annotation emitted by pnpm/action-setup and actions/upload-artifact in the package workflow.

@greptile-apps

greptile-apps Bot commented May 1, 2026

Copy link
Copy Markdown

Greptile Summary

Updates the SHA pin for the tinyland-inc/ci-templates reusable workflow js-bazel-package.yml in both ci.yml and publish.yml from 21e0093 to d202fc0, picking up Node 24 JavaScript action runtimes to resolve the Node.js 20 deprecation warnings emitted by pnpm/action-setup and actions/upload-artifact. Both files are updated to the same commit, keeping them consistent.

Confidence Score: 5/5

Safe to merge — minimal, symmetric change with no logic or configuration impact.

The only change is a SHA bump of a pinned reusable workflow, applied identically to both workflow files. No inputs, permissions, or logic are modified. No issues found.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/ci.yml SHA pin for js-bazel-package.yml reusable workflow bumped from 21e0093 to d202fc0; no other changes.
.github/workflows/publish.yml Same SHA bump as ci.yml; both workflows now pinned consistently to d202fc0.

Sequence Diagram

sequenceDiagram
    participant GH as GitHub Actions
    participant CI as ci.yml / publish.yml
    participant CT as ci-templates (d202fc0)
    participant R as Runner

    GH->>CI: Trigger (PR / push / tag)
    CI->>CT: uses: js-bazel-package.yml@d202fc0
    CT->>R: Setup Node 24 actions (pnpm/action-setup, upload-artifact)
    R->>R: build / test / package / publish
    R-->>GH: Results
Loading

Reviews (1): Last reviewed commit: "ci: use Node 24 reusable package workflo..." | Re-trigger Greptile

@Jesssullivan Jess Sullivan (Jesssullivan) merged commit 8b76bc0 into main May 1, 2026
5 checks passed
@Jesssullivan

Copy link
Copy Markdown
Contributor Author

Merged and verified on main.

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