-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
Unpack all packages? #563
Comments
I would even add a list of packs that will be unpacked and the dependencies that will be resolved, and add a prompt to ask confirmation to do it 👍 And I would even go further: unpack all packs by default when requiring them. Packs kinda "hide" installed dependencies, and they are even considered as "transient dependencies" in the end, because you don't use what's in the pack, you only use what's provided by the pack (that might change one day). Like, what if one day |
I only use packs to install all the things i need. Im unpacking them after that instantly. I don't like the idea of having hidden dependencies in my projects. I like the idea from @Pierstoval to unpack by default. But I would appreciate one of these solution that i don't need to unpack everything manually especially because some packs require packs (amqp-pack requires serializer-pack as an example). Thumbs up from me 👍 |
I agree to review the operation of the packs as well. @Pierstoval, we currently have the case with the orm-pack (https://github.com/symfony/orm-pack/blob/v1.0.7/composer.json) which bump doctrine-bundle to version 2 and there's a lot of open issue associated with it because many projects have been broken by this transparency :/ |
I'm not sure that unpacking should be provided as a helper, but listing all existing packs as a suggestion when no packs are provided on the command line could be nice.
PR welcome, help wanted. |
If the command |
Now that unpacking happens by default, I don't think this will get any tracking. |
Usually I don't care about unpacking packs ... but I was playing with this feature and tried to run this command:
For those of you who usually unpack packages, what do you think about adding support to unpack all packs at once running the
composer unpack
command without arguments? Would that be useful ... or is not a common need to unpack everything? Thanks.The text was updated successfully, but these errors were encountered: