-
Notifications
You must be signed in to change notification settings - Fork 67
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
Update Nestjs dependencies #516
Comments
+1, allow @nestjs/* at version 10 |
Hi @liaoliaots. I submitted a pull request (#517). Please check and update the npm package. Thank you all. |
hey @mahsumurebe, i think the peer dependencies for the packages also need to be bumped individually |
Hey @chunkhang What peer dependencies are you talking about? There are also changes to the pnpm-lock.yaml file. Which ones are missing? |
@mahsumurebe i think these files:
|
Any updates please ? |
bump for dep bump |
It looks like the author has not been keeping up with the repository for some time. I just forked it to https://github.com/Songkeys/nestjs-redis and updated the NestJS dependencies to The npm package is |
Thanks @Songkeys. We're planning to use your package as it unblocks us and allows us to upgrade to Nest v10. Thanks a lot for your efforts. |
Got a package for health as well, by chance? |
@simplenotezy Yes. Try https://www.npmjs.com/package/@songkeys/nestjs-redis-health. |
any updates? |
1 similar comment
any updates? |
Any news? |
Hoping this will be updated. For now, using @Songkeys fork - hopefully that one stays updated.. :) |
Moved to using @Songkeys - thanks |
Any news? |
Another solution for this is to use "overrides": {
"@liaoliaots/nestjs-redis": {
"@nestjs/common": "^10.0.0",
"@nestjs/core": "^10.0.0"
},
"@liaoliaots/nestjs-redis-health": {
"@nestjs/common": "^10.0.0",
"@nestjs/core": "^10.0.0",
"@nestjs/terminus": "^10.0.0"
}
}, |
Thanks a lot!!! It works :) |
Update nestjs dependencies to latest version
The text was updated successfully, but these errors were encountered: