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

support any node 18 lts #22372

Merged
merged 3 commits into from
Jun 2, 2023
Merged

support any node 18 lts #22372

merged 3 commits into from
Jun 2, 2023

Conversation

mshima
Copy link
Member

@mshima mshima commented May 31, 2023

Recent changes to p-transform replaced the implementation that was causing problems at node >= v18.14.0.

Bump node to v18.16.0

Fixes #21380.


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@github-actions github-actions bot added the theme: dependencies Pull requests that update a dependency file label May 31, 2023
@mraible
Copy link
Contributor

mraible commented May 31, 2023

Since https://nodejs.org shows 18.16.0 as the LTS, shouldn't that be our minimum version?

@DanielFran
Copy link
Member

DanielFran commented May 31, 2023

Agree!
Probably we should remove 16 too, the backport should be release soon: https:/nodejs/release#release-schedule
image

@mshima
Copy link
Member Author

mshima commented Jun 1, 2023

node 16 won't have the traditional 18 months of lts maintenance support.
Let's drop node 16 support then.

@DanielFran DanielFran merged commit edc9ede into jhipster:main Jun 2, 2023
@mshima mshima deleted the skip_ci-ptransform branch June 2, 2023 21:01
@DanielFran DanielFran added this to the 8.0.0-beta.1 milestone Jun 3, 2023
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.

Node v18.14.0 breaks jhipster
3 participants