- Issue created by @mxr576
- Merge request !228Add branch alias to help Composer with dependency resolution → (Merged) created by mxr576
- 🇦🇹Austria drunken monkey Vienna, Austria
Thanks for reporting this issue and already providing an MR!
Unfortunately hard to test whether this actually works … How confident are you in your solution? Any examples of other projects where this worked/helped? - 🇭🇺Hungary mxr576 Hungary
I am quite confident, you do not need DDEV to reproduce this issue at all, just clone the project and try to run
composer install
inside the folder. It is going to fail the same way because Composer cannot resolve other dependency versions without the branch alias. - 🇦🇹Austria drunken monkey Vienna, Austria
When running
composer install
from inside the Git checkout of this module you’d also need to add therepositories
entry for the Drupal Packagist repo, but yes, I then also had to add the branch alias for it to work.While I’m again not familiar enough with Ddev to confidently say whether this change works as intended or is even needed, it seems harmless enough and does fix at least something, so it should hopefully at least not cause any problems. I therefore merged it, hopefully it will help a few people.
Thanks again! -
drunken monkey →
committed 4c39aa8d on 8.x-1.x authored by
mxr576 →
Issue #3520657 by mxr576, drunken monkey: Fixed Composer issues due to...
-
drunken monkey →
committed 4c39aa8d on 8.x-1.x authored by
mxr576 →