liaoliaots / nestjs-redis

Redis module for Nest framework (node.js). Support node-redis & ioredis.
MIT License
388 stars 70 forks source link

Update Nestjs dependencies #516

Closed emanuelmcp closed 1 month ago

emanuelmcp commented 1 year ago

Update nestjs dependencies to latest version

pneme commented 1 year ago

+1, allow @nestjs/* at version 10

mahsumurebe commented 1 year ago

Hi @liaoliaots.

I submitted a pull request (#517). Please check and update the npm package.

Thank you all.

chunkhang commented 1 year ago

hey @mahsumurebe, i think the peer dependencies for the packages also need to be bumped individually

mahsumurebe commented 1 year ago

Hey @chunkhang

What peer dependencies are you talking about? There are also changes to the pnpm-lock.yaml file. Which ones are missing?

chunkhang commented 1 year ago

@mahsumurebe i think these files:

heiba commented 1 year ago

Any updates please ?

Distortedlogic commented 1 year ago

bump for dep bump

songkeys commented 1 year ago

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 ^10.0.0.

The npm package is @songkeys/nestjs-redis and can be installed if you're blocked from migrating to NestJS 10 because of it. Once the author comes back, you can change it back to @liaoliaots/nestjs-redis.

heiba commented 1 year ago

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.

simplenotezy commented 1 year ago

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 ^10.0.0.

The npm package is @songkeys/nestjs-redis and can be installed if you're blocked from migrating to NestJS 10 because of it. Once the author comes back, you can change it back to @liaoliaots/nestjs-redis.

Got a package for health as well, by chance?

songkeys commented 1 year ago

Got a package for health as well, by chance?

@simplenotezy Yes. Try https://www.npmjs.com/package/@songkeys/nestjs-redis-health.

fingeromer commented 1 year ago

any updates?

ZhaoKunLong commented 1 year ago

any updates?

andredelima92 commented 1 year ago

Any news?

jacobsandersen commented 1 year ago

Hoping this will be updated. For now, using @Songkeys fork - hopefully that one stays updated.. :)

evyros commented 1 year ago

Moved to using @Songkeys - thanks

awuzi commented 10 months ago

Any news?

mentos1386 commented 9 months ago

Another solution for this is to use overrides in your package.json. Like this:

  "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"
    }
  },
yhjs1211 commented 8 months ago

@mentos1386

Thanks a lot!!! It works :)

liaoliaots commented 1 month ago

Please check out the new version: version 10 of @liaoliaots/nestjs-redis.