Skip to content

build: switch to Node.js 24#423

Merged
DMartens merged 3 commits into
mainfrom
node-24
May 22, 2026
Merged

build: switch to Node.js 24#423
DMartens merged 3 commits into
mainfrom
node-24

Conversation

@mdjermanovic
Copy link
Copy Markdown
Member

Prerequisites checklist

AI acknowledgment

  • I did not use AI to generate this PR.
  • (If the above is not checked) I have reviewed the AI-generated content before submitting.

What is the purpose of this pull request?

Switches build to Node.js 24.

I've already updated settings on Netlify to use Node.js 24, though it should have no effect (it was actually set to 18.x, but Netlify was using 20.x because of .nvmrc).

What changes did you make? (Give an overview)

Updated .nvmrc, package.json and package-lock.json to pin Node.js 24. Also updated README accordingly.

Related Issues

Closes #378.

Is there anything you'd like reviewers to focus on?

Are there any other places that should be updated?

@netlify
Copy link
Copy Markdown

netlify Bot commented May 19, 2026

Deploy Preview for eslint-code-explorer ready!

Name Link
🔨 Latest commit 89d09df
🔍 Latest deploy log https://app.netlify.com/projects/eslint-code-explorer/deploys/6a107efd26592000081ae41f
😎 Deploy Preview https://deploy-preview-423--eslint-code-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@eslintbot eslintbot added this to Triage May 19, 2026
@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage May 19, 2026
@mdjermanovic mdjermanovic added the accepted There is consensus among the team that this change meets the criteria for inclusion label May 19, 2026
@mdjermanovic mdjermanovic moved this from Needs Triage to Implementing in Triage May 19, 2026
Comment thread package.json Outdated
Comment thread README.md Outdated
DMartens
DMartens previously approved these changes May 20, 2026
Copy link
Copy Markdown
Contributor

@DMartens DMartens left a comment

Choose a reason for hiding this comment

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

Changes LGTM, thanks. Leaving open for lumirs suggestion.

Co-authored-by: lumir <rpfos@naver.com>
@DMartens DMartens merged commit f409a97 into main May 22, 2026
9 checks passed
@DMartens DMartens deleted the node-24 branch May 22, 2026 19:35
@github-project-automation github-project-automation Bot moved this from Implementing to Complete in Triage May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion build

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

Suggestion: Update Node.js version support

4 participants