- Issue created by @drunken monkey
- @drunken-monkey opened merge request.
- 🇦🇹Austria drunken monkey Vienna, Austria
Created MR 266 with the latter change, the different namespace import. Anyone, please feel free to go in and implement the rest!
In
📌
Update Drush commands for Drush 12+
Needs review
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.
Active
1.0
Drush / Rules
It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.
Created MR 266 with the latter change, the different namespace import. Anyone, please feel free to go in and implement the rest!