Drush 9 will require Drush services provided by a module to be listed in the extra section of the composer.json

Patch coming.

CommentFileSizeAuthor
#2 default_content-composer_file_needs_drush_services_defined-2932809-2-D8.patch321 bytesPerignon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Perignon created an issue. See original summary.

Perignon’s picture

Perignon CreditAttribution: Perignon commented
FileSize
default_content-composer_file_needs_drush_services_defined-2932809-2-D8.patch321 bytes

Patch. Adds the soon-to-be-required information for Drush 9. Currently warnings are thrown in the console.

Perignon’s picture

Perignon CreditAttribution: Perignon commented
Status: Active » Needs review
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented

Is there links to drush 9 that prove that requirement?

Perignon’s picture

Perignon CreditAttribution: Perignon commented

You can see it in this PR:

https://github.com/drush-ops/drush/pull/3189/files

And other projects like https://www.drupal.org/project/devel/issues/2927090 and https://www.drupal.org/project/config_split/issues/2926474

Perignon’s picture

Perignon CreditAttribution: Perignon commented

Here ya go: http://docs.drush.org/en/master/commands/

Perignon’s picture

Perignon CreditAttribution: Perignon commented

You must not be using Drush 9.x yet. For now, 9.x is only throwing warnings but it will be required going forward at some point.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

Thanx a lot for docs, will commit asap

  • andypost committed a3bc209 on 8.x-1.x authored by Perignon 
    Issue #2932809 by Perignon: Drush 9 services must be listed in composer....
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Reviewed & tested by the community » Fixed

Commited, thanx for contribution!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.