nest-modules / ioredis

:see_no_evil: :hear_no_evil: :speak_no_evil: A ioredis module for Nest framework (node.js)
MIT License
145 stars 31 forks source link

Error when install this package #256

Closed duard closed 11 months ago

duard commented 2 years ago
npm install --save @nestjs-modules/ioredis ioredis
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR! 
npm ERR! While resolving: chatclass-org@0.0.0
npm ERR! Found: @nestjs/common@8.2.5
npm ERR! node_modules/@nestjs/common
npm ERR!   @nestjs/common@"^8.0.0" from the root project
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer @nestjs/common@"^6.7.0 || ^7.0.0" from @nestjs-modules/ioredis@1.0.0
npm ERR! node_modules/@nestjs-modules/ioredis
npm ERR!   @nestjs-modules/ioredis@"*" from the root project
npm ERR! 
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR! 
npm ERR! See /home/duard/.npm/eresolve-report.txt for a full report.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/duard/.npm/_logs/2022-01-19T08_23_03_122Z-debug-0.log
micalevisk commented 2 years ago

the error saids that the required peer deps was not found

https://github.com/nest-modules/ioredis/blob/72372842b35a2694d2c6987d4024f762690fe7a1/package.json#L27-L31

this package doesn't support nestjs v8

micalevisk commented 2 years ago

I guess this can be closed now

https://github.com/nest-modules/ioredis/blob/155a77249a95bcee84b8e87ea101e8588d4e93cb/package.json#L27-L31

ubald commented 2 years ago

It would need to be published to npm so that we can use it.

atassis commented 2 years ago

@juandav can you publish a fix, please?

juandav commented 2 years ago

DONE

atassis commented 2 years ago

@juandav hi there. Can you also add a new version of ioredis (5th is already presented) to the package json and bump the version?

rsathishtechit commented 1 year ago

@juandav Bump the ioredis to v5 to resolve the dependency issue.

juandav commented 11 months ago

The library has already been updated to the latest version.