Description
We are currently building out drush commands but will need to support drush 9. At this time, the drush commands are in progress so I don't want to refer to them by name in fear that they could drastically changes. Whoever does this task should review all drush commands in system_monitor.drush.inc and update them accordingly.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 3094757-4.patch | 10.18 KB | robpowell |
Comments
Comment #2
robpowellComment #3
robpowellComment #4
robpowellhere is the first attempt. To get running in ddev you have to do composer require drush/drush:~9.
I am hoping to get some phpunit tests up to test the new SystemMonitorCommands class.
Comment #5
robpowellComment #6
merauluka commented@robpowell We should consider updating the drush 8 commands to use the drush 9 class-based approach so we only have to maintain a single set of commands.
Comment #7
robpowell@merauluka, I believe this is the class approach. Let me know if there is a different format I should be following. To test you have to update to drush 9: composer require drush/drush:~9.
Comment #8
robpowellComment #9
merauluka commentedA suggestion for this patch would be to avoid backslashes in translated text. It's easier to read if you simply enclose the text in double quotes, which would make escaping the single quotes unnecessary.
See the coding standards doc:
https://www.drupal.org/docs/develop/standards/coding-standards#quotes
Comment #12
merauluka commentedI have committed the changes here with a couple of small updates. It appears a single existence check was calling a non-existent variable.