-
Notifications
You must be signed in to change notification settings - Fork 160
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
Merge in config
/ allow-plugins
#229
Comments
For reference, |
I also just now found this issue. |
I think it makes sense to merge the whole config, just so additional options like |
Right now, suppose there is a root
composer.json
with the following peroperties set:(simplified)
and the following
composer.local.json
:Running
composer update
with this setup will try to install thecomposer/installers
plugin. However, as it is not mentioned in the mainallow-plugins
section, the user will be prompted to decide if this plugin is allowed to be installed or not.The composer-merge plugin should merge in the
allow-plugins
config from included into the main one. However, I'm not quite sure on what level: Should it be only theallow-plugins
property, which probably would be fine to do unconditionally. If it would make sense to consider the whole config property, it would most likely make sense to have amerge-config
(or alike) config for the merge-plugin, like it is already present for the extra and scripts properties.The text was updated successfully, but these errors were encountered: