Skip to content
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

chore(transactional-adapter-mongodb): fix mongodb peer dependency ver… #190

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

JonnyBGod
Copy link
Contributor

…sion

Fixes:

npm error code ERESOLVE
npm error ERESOLVE unable to resolve dependency tree
npm error
npm error While resolving: datagatherers-backend@0.0.0
npm error Found: mongodb@6.11.0
npm error node_modules/mongodb
npm error   mongodb@"^6.11.0" from the root project
npm error
npm error Could not resolve dependency:
npm error peer mongodb@"> 6" from @nestjs-cls/transactional-adapter-mongodb@1.1.3
npm error node_modules/@nestjs-cls/transactional-adapter-mongodb
npm error   @nestjs-cls/transactional-adapter-mongodb@"*" from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.
npm error

@Papooch Papooch self-requested a review December 5, 2024 17:23
Copy link
Owner

@Papooch Papooch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this was a silly overlook :)

@Papooch
Copy link
Owner

Papooch commented Dec 5, 2024

You'll need to commit the update lockfile, too, though

@JonnyBGod
Copy link
Contributor Author

Ups sorry did a quick fix through github. will do it properly in a few minutes.

@Papooch Papooch enabled auto-merge (squash) December 5, 2024 21:49
@Papooch Papooch merged commit 8bf4341 into Papooch:main Dec 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants