-
-
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
Add a progress bar to the queue:run command #5986
base: 13.x
Are you sure you want to change the base?
Add a progress bar to the queue:run command #5986
Conversation
If the queue worker logs messages, are they still shown? If so, I think this LGTM |
I just tested and it looks like Drupal logs are never logged to the console: not with the progress bar, not without. |
Thanks. I'm also curious about any drush log messages that come from the worker. Maybe thats too uncommon to care about. The screenshot should also show the total number of items, right? Otherwise progress is a bit hard to understand. Once that is done, we could arguably get rid of the log message that reports progress. |
It does, except when there's 0 items to be processed, which is why I proposed to not even start the progress bar at all in that case. |
Thats super ugly IMO. Once a progress bar starts repeating lines it loses its value IMO. Happy to discuss more. |
I agree, but I'm not sure if there's anything straightforward we can do about this. Also haven't tested this yet with Laravel Prompts https://laravel.com/docs/11.x/prompts#progress |
On the other hand, progress bars are also used by the |
Right, there isnt. a great alternative. And thats why this command hasnt been made a progress bar for years. The other two commands you mention were converted by eager contributors, not me. I merged their PRs because others liked it. I actually dislike progress bars on CLI personally. |
f540485
to
92eab9a
Compare
92eab9a
to
06e36dd
Compare
Just an idea for a small improvement I had: the queue:run command could use a progress bar, especially in case of large queues.