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

Deploy command doesn't copy devDependencies #4992

Closed
ariesclark opened this issue Jul 7, 2022 · 4 comments
Closed

Deploy command doesn't copy devDependencies #4992

ariesclark opened this issue Jul 7, 2022 · 4 comments

Comments

@ariesclark
Copy link

pnpm version: 7.5.0

Code to reproduce the issue:

# Inside apps/api/dist, run the following command.
pnpm deploy --filter \".\" apps/api/dist

Expected behavior:

Deploy would copy all dependencies, including devDependencies.

Actual behavior:

Only packages inside dependencies are included in the output node_modules.

Additional information:

  • node -v prints: v18.2.0
  • Windows, macOS, or Linux?: Ubuntu 22.04 LTS x86_64
@ariesclark
Copy link
Author

Also, the resulting directory doesn't contain a pnpm-lock.yaml, but that's not relevant to this particular issue.

@robations
Copy link

I've not tried this feature, but I would have guessed that devDependencies are only needed for development, not to be deployed?

@kyle-mccarthy
Copy link

FYI I had a similar problem in #5035, and #5040 fixes the devDependencies issue.

@ariesclark
Copy link
Author

fixed in #5040

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants