Problem/Motivation

Drush is a great tool but it is not required to make this module work. It shouldn't be declared as a dependency in the composer.json file.
If necessary for the tests, it should be declared as a dev dependency.
It could also be added to the suggests section to inform the persons requiring it that it has drush commands.

Proposed resolution

Remove drush from composer.json's dependencies.

Issue fork schemadotorg-3565234

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

duaelfr created an issue. See original summary.

duaelfr opened merge request !284

duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented
Status: Active » Needs review
jrockowitz’s picture

jrockowitz commented

Composer is failing to build.

I think Drush should be declared as a dev dependency.

jrockowitz’s picture

jrockowitz commented

There seems to be an unrelated composer issue occurring.

jrockowitz’s picture

jrockowitz commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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