-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
devDependencies are not installed when both NODE_ENV=production
and NPM_CONFIG_PRODUCTION=false
are set
#1975
Comments
Is that a real thing? I didn't come across it in any documentation, and Googling for it is nearly impossible due to the articles about yarn production that show up... |
@adamreisnz Yeah, it's the same deal with |
Ok thanks, I'll give it a go. Might be useful for my issue where I want to install only prod dependencies, yet have Edit ok it's not useful after all, as it overwrites |
BTW a PR for the docs change is welcome
…On Sat, 17 Dec 2016 at 21:02, Adam Reis ***@***.***> wrote:
Ok thanks, I'll give it a go. Might be useful for my issue where I want to
install *only* prod dependencies, yet have NODE_ENV set to something
else, like staging.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#1975 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ACBdWHhSC4nRIHLurCnx3u7WhVaHTCuHks5rJE32gaJpZM4K41NQ>
.
|
This issue is blocking me from migrating to yarn. When can we expect a release including the fix? |
Do you want to request a feature or report a bug?
bug
What is the current behavior?
NODE_ENV
seems to be taking precedence overNPM_CONFIG_PRODUCTION
(andYARN_PRODUCTION
).In npm, it's the opposite.
If the current behavior is a bug, please provide the steps to reproduce.
This doesn't install
blob
.What is the expected behavior?
It should install
devDependencies
like npm does:This does install
blob
.Please mention your node.js, yarn and operating system version.
Windows 10
The text was updated successfully, but these errors were encountered: