-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make MessengerCommands the default w/ an option #4063
Comments
It's a commandfile so it loads on every request. You don't need to extend it - it logs all Drupal messages to the drush log. The disclaimer is that it only works after Drupal has fully bootstrapped. So it misses messages during cache rebuild and site install too. |
I really don't understand anything Drush does these days. Where could I read more? |
This commandfile is implementing hooks provided by Annotated Command project. See https://github.com/consolidation/annotated-command/blob/master/README.md. I'm also happy to discuss via chat or voice. |
Well then let's continue in consolidation/annotated-command#187 |
#3489 is awesome. There are no articles, examples or tests so I'd wager noone knows its there. It's not easy to find. Why not move the functionality to
DrushCommands
triggered with an option? If I had any idea on how to take options inMessengerCommands::log()
(orpre/post
) I'd write it myself. Ie. if you like the idea and tell me how, I will code this.The text was updated successfully, but these errors were encountered: