Skip to content

Node 24#174

Merged
athomas merged 4 commits intodart-lang:mainfrom
matthewnitschke-wk:node_24
Mar 19, 2026
Merged

Node 24#174
athomas merged 4 commits intodart-lang:mainfrom
matthewnitschke-wk:node_24

Conversation

@matthewnitschke-wk
Copy link
Copy Markdown
Contributor

  • Thanks for your contribution! Please replace this text with a description of what this PR is changing or adding and why, list any relevant issues, and review the contribution guidelines below.

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.

This PR is the fix for the issue I reported here: #173

  • Updates the action to run on node v24
  • Adds @vercel/ncc as a dev dependency for easier npm run dist execution
  • Adds "engines" for node v24 to ensure future usage of node v24
Contribution guidelines:

Many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

Note: The Dart team is trialing Gemini Code Assist. Don't take its comments as final Dart team feedback. Use the suggestions if they're helpful; otherwise, wait for a human reviewer.

Comment thread dist/main.cjs Outdated
@@ -1,4 +1,4 @@
// Generated by dart2js (NullSafetyMode.sound, csp, intern-composite-values), the Dart to JavaScript compiler version: 3.8.0-91.0.dev.
// Generated by dart2js (, csp, intern-composite-values), the Dart to JavaScript compiler version: 3.9.4.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Regenerated with dart 3.9.4 because thats what I happened to have on my computer at the time

I'm happy to revert this back to dart 3.8.0, or bump this up to 3.11.2 if thats preferred

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.

I think latest stable is ideal here. @athomas should 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.

Yes, please bump it to 3.11.2.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated!

@matthewnitschke-wk matthewnitschke-wk marked this pull request as ready for review March 13, 2026 18:06
@athomas athomas merged commit 6e0ff0b into dart-lang:main Mar 19, 2026
29 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.

3 participants