-
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
Add support for passing original command-line options to installer #169
base: master
Are you sure you want to change the base?
Conversation
previously used events are not triggered.
Can somebody take a look at the test build in the Travis CI who has admin right? Test jobs were not even started. Maybe @bd808? |
Hey, |
Thanks for this. I found this PR when looking for a fix for --ignore-platform-reqs. I am going to pull in this as a patch and see if it works. |
ping :D forwarding --no-suggest would be 💯 |
ping @bd808, you are the maintainer, aren't you? |
Is this PR ongoing or dead? I'm in need of |
It is dead on my part, someone can take over. We stopped using the merge plugin because it proved to be a bottleneck several times. We rather rely on OTTB Composer features, e.g.: use path repositories on the project level to register dependencies of custom components when needed; and we developed our own upstream Composer repository for shared components. AFAIK, this covers everything that anyone used/using the merge plugin. It was a great tool back in the days but personally, I do not think it is necessary in 2021. |
Thanks for the update. This was very helpful. After re-evaluating our use-case we also decided to drop this. |
@reedy you the only maintainer left? This PR needs some love. 💚 If I run |
We probably should start a new MR for it to work around the conflicts AND make it respect |
Merge plugin did not respected
--prefer-lowest
option so highest-lowest testing could not be achieved with it. This PR fixes this problem and also fixes some others: