ENG-1805 - Update Node.js version to Node 22#1149
Conversation
… project - Changed Node.js engine requirement in package.json to >=22. - Updated @types/node dependencies to version 22.20.0 in various package.json files. - Adjusted CI workflows to use Node.js version 22 for all relevant jobs. - Ensured compatibility with Node 22 in pnpm-lock.yaml and pnpm-workspace.yaml. - Minor comment update in route.ts to clarify groupBy functionality in relation to Node.js versions.
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
|
@codex review |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
maparent
left a comment
maparent
left a comment
There was a problem hiding this comment.
LGTM. I noticed the action runs still used node 20, but that was a caching artifact cleared with a re-run.
I would suggest adding a .nvmrc : lts/jod at the root, so we all use the relevant version, but not sure how you feel about nvm.
2 participants
Uh oh!
There was an error while loading. Please reload this page.