Skip to content

chore: update node.js to v26#38285

Open
silverwind wants to merge 1 commit into
go-gitea:mainfrom
silverwind:bump-nodejs-26
Open

chore: update node.js to v26#38285
silverwind wants to merge 1 commit into
go-gitea:mainfrom
silverwind:bump-nodejs-26

Conversation

@silverwind

@silverwind silverwind commented Jun 30, 2026

Copy link
Copy Markdown
Member
  • bump ci, flake and @types/node to node 26
  • regenerate flake.lock which is needed for that package
  • refactor workflow to use shared composite action

Bump the CI/build and dev-shell Node.js toolchain to v26, refresh
@types/node to match, and refactor the release workflows to reuse the
existing node-setup composite action instead of duplicating the
pnpm + setup-node block (which previously required editing node-version
in several coordinated spots). engines.node floor is left unchanged.

Assisted-by: Claude:Opus 4.8
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jun 30, 2026
@github-actions github-actions Bot added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Jun 30, 2026
@silverwind silverwind requested a review from Copilot June 30, 2026 17:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the repository’s Node.js toolchain to v26 across local development (Nix flake), frontend type dependencies, and GitHub Actions CI/release workflows, and centralizes Node/pnpm setup logic into a shared composite action.

Changes:

  • Bump @types/node to 26.0.1 and regenerate pnpm-lock.yaml to reflect the updated Node types and transitive dependencies.
  • Update the Nix dev shell to use pkgs.nodejs_26 and refresh flake.lock to a nixpkgs revision that supports the new toolchain.
  • Refactor release workflows to use the shared .github/actions/node-setup composite action; update remaining direct setup to Node 26 where applicable.

Reviewed changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pnpm-lock.yaml Regenerates lockfile to align with @types/node@26.0.1 and updated transitive deps (e.g., undici-types).
package.json Bumps @types/node devDependency to 26.0.1.
flake.nix Switches dev shell Node toolchain from nodejs_24 to nodejs_26.
flake.lock Updates pinned nixpkgs revision/hash to match the regenerated flake inputs needed for the Node bump.
.github/workflows/release-tag-version.yml Uses the shared node-setup composite action instead of inline Node/pnpm setup.
.github/workflows/release-tag-rc.yml Uses the shared node-setup composite action instead of inline Node/pnpm setup.
.github/workflows/release-nightly.yml Uses the shared node-setup composite action instead of inline Node/pnpm setup.
.github/workflows/pull-labeler.yml Updates actions/setup-node to Node 26 for PR title/label tooling.
.github/actions/node-setup/action.yml Introduces/updates a composite action to standardize pnpm + Node 26 setup (with optional pnpm caching).
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bircni bircni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe we could use a .node-version file?

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/need 1 This PR needs approval from one additional maintainer to be merged. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants