In Drush 12, Drush commands are expected to be put into src/Drush/Commands/ subdirectory. See https://www.drush.org/12.x/commands/

In #3372643: Drush 12 support put them into src/Drush/ directly, leaving the appending /Commands/ directory out.

CommentFileSizeAuthor
#2 3403792-2.patch1.61 KBnorman.lol

Comments

leymannx created an issue. See original summary.

norman.lol’s picture

norman.lol
he/him
Primary language German
Location Berlin
commented
StatusFileSize
new 3403792-2.patch1.61 KB
norman.lol’s picture

norman.lol
he/him
Primary language German
Location Berlin
commented
Title: Move Drush commands to src/Drush/Commands/ » Move Drush commands from src/Drush/ to src/Drush/Commands/
heddn’s picture

heddn
Primary language English
Location Illinois
commented

I've manually tested drush 12.4.3 and drupal core 11.x and migrate tools 6.0.x. I had no errors. Also, tests also pass. What are you seeing to indicate that something isn't working with Drush 12?

moshe weitzman’s picture

moshe weitzman commented

Thanks for testing. Its true that we currently discover commands src/Drush. However the documented place to put them is src/Drush/Commands. See "Drush 12 expects all commandfiles in the /Drush/ directory" at https://www.drush.org/12.x/commands/. So for optimum future compat, you would move them as this issue suggests. In practice its not likely to matter.

norman.lol’s picture

norman.lol
he/him
Primary language German
Location Berlin
commented
Status: Active » Needs review

  • heddn committed ec3d1ebb on 6.0.x authored by leymannx 
    Issue #3403792 by leymannx, heddn, moshe weitzman: Move Drush commands...
heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Needs review » Fixed

Thanks for the clarity and suggestions.

Status: Fixed » Closed (fixed)

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