Skip to content

AB#532 - Update to node v24#5739

Merged
vesameskanen merged 3 commits intov3from
update-to-node-v24
Apr 16, 2026
Merged

AB#532 - Update to node v24#5739
vesameskanen merged 3 commits intov3from
update-to-node-v24

Conversation

@VillePihlava
Copy link
Copy Markdown
Contributor

Proposed Changes

  • Update node to v24.14.1
  • I tested running the UI and unit tests and it worked
  • I removed an old node14-patch

Problems

  • A lot of new warnings
  • babel-esm-loader.mjs is workaround for mocha ESM syntax problems
  • Not everything has been tested so some things might break in the future

@VillePihlava VillePihlava marked this pull request as draft April 10, 2026 08:19
Copy link
Copy Markdown
Member

@vesameskanen vesameskanen left a comment

Choose a reason for hiding this comment

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

Seems to work fine. I am surprised that leaflet patch can be removed now.

Comment thread test/unit/helpers/babel-esm-loader.mjs Outdated
const transformed = await transformAsync(source, {
filename: fileURLToPath(url),
caller: {
name: 'mocha-esm-loader',
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.

'babel-esm-loader'

@VillePihlava VillePihlava marked this pull request as ready for review April 16, 2026 09:30
@VillePihlava VillePihlava changed the title Update to node v24 AB#532 - Update to node v24 Apr 16, 2026
@vesameskanen vesameskanen merged commit 4ad6fe1 into v3 Apr 16, 2026
7 checks passed
@vesameskanen vesameskanen deleted the update-to-node-v24 branch April 16, 2026 10:08
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