-
Notifications
You must be signed in to change notification settings - Fork 68
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
refactor: update nestjs dependencies #517
base: main
Are you sure you want to change the base?
Conversation
Any estimate for merging this? My team is excited for SWC support in v10 Nest.js for which this bump is needed 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any news ?
Hey @liaoliaots ! Any updates? |
@mahsumurebe, I think this PR should also update the requiremenets in the readme for Nest and Node |
any news? it has been some time since last commit is this maintained? @liaoliaots |
Hi @liaoliaots Necessary improvements have been made. Can you check the pull request? |
@liaoliaots Would much appreciate your support on this. We're kind of blocked on upgrading to Nest.js v10 because of the nestjs-redis not supporting it. Anything I can do from my side, I will be glad to support. I thank you for your efforts on this project. |
Hi @liaoliaots Can you check the pull request? |
it's been close to 2 months now, is it time to create a fork instead? |
It's already created by @Songkeys |
It is possible to override dependencies in
|
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: #516
What is the new behavior?
Does this PR introduce a breaking change?
Other information