Problem/Motivation

When using Drush 10.5.0 on a site with advancedqueue installed, I get the following message:

 [info] advancedqueue should have an extra.drush.services section in its composer.json. See http://docs.drush.org/en/10.x/commands/#specifying-the-services-file.

... the link there doesn't work - rather it is https://www.drush.org/latest/commands/#specifying-the-services-file

I get the sense that Drush 10 wants module developers to register any Drush services file(s) they define in composer.json, along with the version of Drush they're supposed to work with.

Proposed resolution

Add a composer.json. Add the following section:

  "extra": {
    "drush": {
      "services": {
        "drush.services.yml": "^9 || ^10"
      }
    }
  }

Remaining tasks

  1. Write a patch
  2. Review and feedback
  3. RTBC and feedback
  4. Commit
  5. Release

User interface changes

None.

API changes

None.

Data model changes

None.

Comments

mparker17 created an issue. See original summary.

mparker17’s picture

Status: Active » Needs review
StatusFileSize
new824 bytes

Here's a patch that adds a minimal composer.json.

mparker17’s picture

Issue summary: View changes
jcnventura’s picture

Status: Needs review » Needs work

Please remove the drupal/core requirement. That is added by the drupal.org packagist from the information on the module's .info.yml file.

mparker17’s picture

Status: Needs work » Needs review
StatusFileSize
new756 bytes
new375 bytes

Certainly! Here's an updated patch.

Thanks!

jcnventura’s picture

Status: Needs review » Reviewed & tested by the community

  • mparker17 authored 9ad582d on 8.x-1.x
    Issue #3225734 by mparker17, jcnventura: Add an extra.drush.services...
jcnventura’s picture

Status: Reviewed & tested by the community » Fixed
mparker17’s picture

Awesome, thanks @jcnventura!

Status: Fixed » Closed (fixed)

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