-
Notifications
You must be signed in to change notification settings - Fork 5
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
NestJS v10 support #207
base: master
Are you sure you want to change the base?
NestJS v10 support #207
Conversation
Add support for NestJS10 by extending the peer dependencies, add support for NodeJS v20
Updated ci to test with new versions of NestJS and Node v20
Hi @toondaey. Please let me know if there is anything missing. |
Thanks for the contribution @EdgarHarutyunyan02 ! I'm sorry for the late response. I'll need some time to make some changes to the entire package. Have some plans but haven't had time to look into it. |
Hi @toondaey, is it going to be a whole refactor? |
Not essentially. Just to update how the way the package is published to npm. I've had to do it manually ever since I wrote the package. |
Ok, that's fine. "overrides": {
"@t00nday/nestjs-got": {
"@nestjs/common": "$@nestjs/common"
}
}, |
Hi all,
With these changes I'm adding support for NestJS v10 which was released in June 2023.
Recently NodeJS v20 transitioned to LTS status, so added that in the supported engines as well.
Updated the CircleCI config to include these new versions as well.
Besides the integration tests I also tested it locally on a NestJS v10 project running with Node v20 and it worked as expected.