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.

Sign up for GitHub

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

Jump to bottom

pnpm install --production #54

Closed
rstacruz opened this issue Jan 31, 2016 · 4 comments
Closed

pnpm install --production #54

rstacruz opened this issue Jan 31, 2016 · 4 comments
Labels
help wanted
Milestone
v1.0.0

Comments

@rstacruz
Copy link
Member

From npm help install:

With the --production flag (or when the NODE_ENV environment variable is set to production), npm will not install modules listed in devDependencies.
@davej
Copy link
Contributor

davej commented Feb 1, 2016

Working on this.

@davej davej mentioned this issue Feb 1, 2016
Merged
@zkochan zkochan modified the milestone: v1.0.0 Aug 18, 2016
zkochan pushed a commit that referenced this issue May 11, 2018
@renovate @zkochan
fix(deps): update dependency package-store to ^0.18.0 (#54)
ae897e5 
* fix(deps): update dependency package-store to ^0.18.0

* fix: types
@etamponi
Copy link
Member

Running:

NODE_ENV=production pnpm install

has no effect, devDependencies are installed anyway.

@etamponi etamponi reopened this Jul 12, 2018
@zkochan
Copy link
Member

zkochan commented Jul 12, 2018

you probably mean that the tarballs are downloaded anyway. If there is no shrinkwrap.yaml file then it will happen because to create the shrinkwrap.yaml file the real manifests are needed from inside the tarballs.

@zkochan zkochan closed this as completed Aug 25, 2018
@zkochan
Copy link
Member

zkochan commented Aug 25, 2018

sorry. Reproducible

@zkochan zkochan reopened this Aug 25, 2018
zkochan added a commit to pnpm/npm-conf that referenced this issue Aug 25, 2018
@zkochan
fix: don't override internal defaults
aaa60eb 
ref pnpm/pnpm#54
zkochan added a commit that referenced this issue Aug 25, 2018
@zkochan
test: fix --production tests
52ce34c 
ref #54
zkochan added a commit that referenced this issue Aug 25, 2018
@zkochan
test: fix --production tests
050c38f 
ref #54
@renovate renovate bot mentioned this issue Oct 27, 2022
Closed
1 task
@renovate renovate bot mentioned this issue Nov 22, 2022
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted
Projects
None yet
Milestone
v1.0.0
Development

No branches or pull requests
4 participants
@davej @rstacruz @etamponi @zkochan