-
Notifications
You must be signed in to change notification settings - Fork 14
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
Drop dev dependencies #51
Comments
Well, that's why they are called dev dependencies! People should install bazaar with Bazaar itself already runs composer in production mode. (It's not clearly visible because it's the default behavior; in the official composer CLI the absence of the |
I meant that bazaar should install packages without including dev dependencies by default. I wasn't specifically talking about installing bazaar 😁 |
It's not working then? It should 😅 |
It's not about dist mode, it's about dev mode: https://github.com/composer/composer/blob/master/src/Composer/Command/InstallCommand.php#L105 |
Oh that's what you said too, but doesn't composer set dev mode if left out.. Seems like it doesn't. We'd have to confirm this anyway. |
Anyone using bazaar most likely does not need any dev dependencies, let's not install those in our vendor.
The text was updated successfully, but these errors were encountered: