Greg Anderson
Greg Anderson
c.f. #4919 Tests are failing.
We have https://github.com/drush-ops/drush/pull/4919 and https://github.com/drush-ops/drush/pull/5136 for Drush 8. Seems like an update of the signatures for the Psysh method overrides and an update to composer.json should make it work again....
@weitzman Drush launcher doesn't support aliases, and it seems that adding alias support there isn't gaining traction. Instead of removing redispatchToSiteLocal, maybe instead we should only simplify the bootstrap logic,...
I think we could consider a backport of #2387 to the 8.x branch if it was opt-in, e.g. if a Drush config option could be set to control whether `defaults-extra-file`...
The general idea here is that we find command classes by discovery, like we used to do (and still do for site-wide commands). We could also consider discovering command classes...
[Symfony containers](https://github.com/symfony/symfony/blob/5.x/src/Symfony/Component/DependencyInjection/Container.php) implement ContainerInterface, which is the requirement for deletage() in league / container, so I think that maybe we could delegate to the Drupal container from the Drush container....
Hm, yeah, that might work.
This feature was untested. Removing it did not break anything else.
Ready for review, but should only be merged into a 12.x branch.
Yeah, the reference was added here: https://github.com/drush-ops/drush/pull/5108/files#diff-fcb54a74a331c6112bd5cf5e32f08a4d0a39f42c16833b899b4fac0339d453c7R204 Wasn't caught because a global install of Drush 11 is not a recommended configuration. Adding a direct dependency on symfony/http-kernel is reasonable, though.