In #3470199: Update Drush commands for Drush 12+ we adapted our Drush integration to be compatible with Drush 12+, by adding attributes as a complement to the existing annotations (which will no longer work in the future).
However, for some reason we missed the \Drush\Attributes\Option attributes which means that our options are undocumented in Drush 12+.
We should fix this by just going over all commands and copying the text from the @option annotations to Option attributes (probably right above the Help attribute, after any Argument attribute).

While we’re at it, it would be great to go back to the old way of just importing Drush\Attributes as CLI and then using #[CLI\…] for the attributes. This was accidentally changed as an unrelated side effect of this merge request. I actually reverted this before merging, but it seems this didn’t get included in the merge after all.

Issue fork search_api-3543806

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:

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

Created MR 266 with the latter change, the different namespace import. Anyone, please feel free to go in and implement the rest!

drunken monkey’s picture

Status: Active » Needs review
Issue tags: -Novice
drunken monkey’s picture

Status: Needs review » Fixed

Merged.

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

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

Maintainers, please credit people who helped resolve this issue.

  • drunken monkey committed 782017fd on 8.x-1.x
    [#3543806] feat: Added Option attributes to our Drush commands.
    
    By:...

Status: Fixed » Closed (fixed)

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