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

Move all @nestjs packages to peerDependencies #28

Merged
merged 3 commits into from
Mar 12, 2022
Merged

Move all @nestjs packages to peerDependencies #28

merged 3 commits into from
Mar 12, 2022

Conversation

queicherius
Copy link
Contributor

Nest v8 no longer stringifies class names but rather uses class references to connect injectables. That means all Nest packages should be defined as peer dependencies in case you want to share them (instead of having each package with its own copy of Nest deps).

Originally posted in nestjs/nest#7568 (comment)


Fixes #19

@MetinSeylan
Copy link
Owner

Hi @queicherius thanks for the pr, I will check it ASAP.

@MetinSeylan MetinSeylan self-assigned this Mar 10, 2022
@MetinSeylan MetinSeylan merged commit 4debd4d into MetinSeylan:main Mar 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build Issues when use pnpm
2 participants