Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
» Replace drush_verbose() and drush_print() calls with drush_log()
Status:
Needs review
» Active
Committed the patch. I also removed drush_verbose().
Any chance you can repace drush_print() too? These will need a 'success' or 'message' or some other type as second param to drush_log since they should print regardless of verbose level.
There are times when we really do want to just print instead of log - mostly when the output could conceivable be consumed by another script. Example: `drush connect` (a new command)
Comments
Comment #1
dawehnerhere is a patch for the HEAD version
Comment #2
moshe weitzman commentedCommitted the patch. I also removed drush_verbose().
Any chance you can repace drush_print() too? These will need a 'success' or 'message' or some other type as second param to drush_log since they should print regardless of verbose level.
Comment #3
moshe weitzman commentedThere are times when we really do want to just print instead of log - mostly when the output could conceivable be consumed by another script. Example: `drush connect` (a new command)
Comment #4
moshe weitzman commented