Skip to content

feat: update to node 22 #3337

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

Open
wants to merge 11 commits into
base: v11.0.0
Choose a base branch
from
Open

feat: update to node 22 #3337

wants to merge 11 commits into from

Conversation

eablack
Copy link
Contributor

@eablack eablack commented Jul 16, 2025

W-18293846

Description

This updates v11 to be compatible with both node 20 and node 22. It will ship with node 22. This also updates eslint to v8, and brings numerous formatting changes to files to satisfy updated rules. I did limit the number of changes required by switching many rules to warn.

Testing

You may need to add a new version of node to your asdf before testing.

  1. Pul this branch down and yarn install and yarn build
  2. Run one of the available commands, like ./bin/run whoami

@eablack eablack temporarily deployed to AcceptanceTests July 17, 2025 15:09 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests July 17, 2025 15:09 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests July 17, 2025 15:09 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests July 17, 2025 15:09 — with GitHub Actions Inactive
@eablack eablack marked this pull request as ready for review July 17, 2025 15:28
@eablack eablack requested a review from a team as a code owner July 17, 2025 15:28
Copy link
Contributor

@sbosio sbosio left a comment

Choose a reason for hiding this comment

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

LGTM! Tried a few of the main commands: whoami, apps, addons and they worked correctly.

eablack added 10 commits July 17, 2025 17:21
* Comment out tests

* Upgrade to oclif/core v4, comment out most commands, libs, and tests

* move to esm

* Update dev/run commands, convert tests to esm

* Make tests compatible with ESM

* revert log-displayer to commonjs

* Uncomment regions, add some tests back

* Update yarn.lock

* Comment out new commands from pg upgrade

* Adjust tests to account for variable table length

* Fix regions test, add stuff back to bin/run and update bin/dev

* fix lint

* Remove temp script

* Update heroku-cli/command

* Add esm ts-node option to tsconfig.json

* Update requires to imports, switch to cli-progress from smooth-progress

* Update to modern module resolution

* Rewrite Editor to stop using ancient library

* Update a few packages

* Remove shellescape, fix lint

* Update to esm version of heroku-cli/command
@eablack eablack temporarily deployed to AcceptanceTests July 18, 2025 21:16 — with GitHub Actions Inactive
@eablack eablack deployed to AcceptanceTests July 18, 2025 21:16 — with GitHub Actions Active
@eablack eablack temporarily deployed to AcceptanceTests July 18, 2025 21:16 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests July 18, 2025 21:16 — with GitHub Actions Inactive
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.

2 participants