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 Node v20 #333

Merged
merged 2 commits into from
Dec 18, 2023
Merged

Support Node v20 #333

merged 2 commits into from
Dec 18, 2023

Conversation

itsikcircle
Copy link
Contributor

@itsikcircle itsikcircle commented Nov 30, 2023

Node v20.10.0 is the latest LTS, many projected that use this as dependency is blocked by the limitation of the node engine to v18 and below.

In this PR I'm adding the needed support for node v20 as it is supported out of the box.

@CLAassistant
Copy link

CLAassistant commented Nov 30, 2023

CLA assistant check
All committers have signed the CLA.

package.json Show resolved Hide resolved
@itsikcircle
Copy link
Contributor Author

cc @filfreire @dimitropoulos could you please help in review this PR?

@itsikcircle
Copy link
Contributor Author

@erunion @jackkav Can you please help in merging this change? Thank you!

@erunion
Copy link
Contributor

erunion commented Dec 13, 2023

@itsikcircle Sorry, I'm just a frequent contributor and don't have any repository permissions here.

@jackkav jackkav merged commit 7c3d401 into Kong:master Dec 18, 2023
4 checks passed
@itsikcircle
Copy link
Contributor Author

Thank you @jackkav for merging this PR. Is it already published?

@itsikcircle
Copy link
Contributor Author

@jackkav How can we get a new version including this change?

@itsikcircle
Copy link
Contributor Author

@jackkav who should I reach out to publish new npm version including this change?

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.

4 participants