Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update to Node 22.13 #4351

Merged
merged 3 commits into from
Jan 14, 2025
Merged

chore: update to Node 22.13 #4351

merged 3 commits into from
Jan 14, 2025

Conversation

marcelgerber
Copy link
Member

@marcelgerber marcelgerber commented Dec 22, 2024

Resolves #4085.

I have been running my local Node setup on Node 22 for several months now, and have not noticed any incompatabilities. I'm pretty certain that this upgrade won't cause us any weird and unexpected issues.

Steps for devs

  • Install Node 22, and possibly make it the default on your machine
    • e.g. if you're using nvm: nvm install 22 && nvm alias default 22

Steps to deploy this

  • Create a PR in https://github.com/owid-grapher/ops to change the install scripts to install node 22, and enable corepack
  • Manually (?) upgrade some non-ephemeral machines
  • It's probably fine to first merge the ops PR, and then merge this PR sometime later - everything should continue to work just fine, even before this here PR

Other notes

The whatwg-url update in here resolves a deprecation notice shown in Node (e.g. when running yarn startAdminServer). It has already been cherry-picked into master: d936a32.

(node:730) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
    at node:punycode:3:9
    at BuiltinModule.compileForInternalLoader (node:internal/bootstrap/realm:399:7)
    at BuiltinModule.compileForPublicLoader (node:internal/bootstrap/realm:338:10)
    at loadBuiltinModule (node:internal/modules/helpers:112:7)
    at Function._load (node:internal/modules/cjs/loader:1100:17)
    at TracingChannel.traceSync (node:diagnostics_channel:322:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:219:24)
    at Module.require (node:internal/modules/cjs/loader:1340:12)
    at require (node:internal/modules/helpers:138:16)
    at Object.<anonymous> (/Users/marcel/Code/owid-grapher-2/node_modules/whatwg-url/lib/url-state-machine.js:2:18)

Copy link

github-actions bot commented Jan 6, 2025

This PR has had no activity within the last two weeks. It is considered stale and will be closed in 3 days if no further activity is detected.

@github-actions github-actions bot added the stale label Jan 6, 2025
@marcelgerber marcelgerber force-pushed the node-22 branch 2 times, most recently from 0a664e3 to 14bedb5 Compare January 7, 2025 13:17
@owidbot
Copy link
Contributor

owidbot commented Jan 7, 2025

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-node-22

SVG tester:

Number of differences (default views): 0 ✅
Number of differences (all views): 0 ✅

Edited: 2025-01-07 13:24:36 UTC
Execution time: 1.33 seconds

@marcelgerber marcelgerber changed the title chore: update to Node 22, enable corepack to manage yarn versions chore: update to Node 22 Jan 14, 2025
@marcelgerber marcelgerber marked this pull request as ready for review January 14, 2025 07:08
@marcelgerber marcelgerber merged commit 53e97fe into master Jan 14, 2025
22 checks passed
@marcelgerber marcelgerber deleted the node-22 branch January 14, 2025 09:50
@marcelgerber marcelgerber changed the title chore: update to Node 22 chore: update to Node 22.13 Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Node.js to 22.x
2 participants