Add vagrant transport to Drush. #4036
Conversation
This allows executing drush calls via vagrant ssh -c by just using vagrant: in the site alias file. Support for this was added in site-process here: consolidation/site-process#40
|
Hm, we really should have had Drush call some Site Process API to add the default transports rather than wire it up manually via Dependency Injection. This way fully specifies what our runtime class usage is, but it's kind of unfortunate to have to modify Drush for every new transport. Maybe there won't be many more, though. This way is therefore okay, once it has passing tests. |
|
Oh, if we use the new vagrant transport, we need to release site process so we can update composer.json here. Let me try your suggestion, that way we could move forward with this PR. |
|
Yeah, this looks much better. I should have done it like this in the first place. I'm leaving shortly for the airport (DrupalCon), but I'll tag a new release for site-process in the near future. |
* master: Remove latest unstable chiclet. Its not helpful. Remove mentions of Shippable. (drush-ops#4079) Add mysql service to Travis config. (drush-ops#4078) Fix link in docs. More docs about dynamic users in site.yml files. Fix drush-ops#4055. Sort inside namespaces in list command. (drush-ops#4057) Print class name in drush_op() Update dependencies. (drush-ops#4051) UNISH_NO_TIMEOUTS is unused since we moved to CLITrait. (drush-ops#4047) Add jsonapi command (drush-ops#4045) Fixes drush-ops#4034: Ensure that status code is nonzero on invalid shutdown (drush-ops#4037) Add webflo/drupal-core-require-dev (drush-ops#4041) Fix TTY detection for runserver command. (drush-ops#4033) Add vagrant transport to Drush. (drush-ops#4036)
This allows executing drush calls via vagrant ssh -c by just using vagrant: in the site alias file. Support for this was added in site-process here: consolidation/site-process#40