-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Change 'Processing' to 'Compressing' in terminal video output #26657
Comments
@jennifer-shehane could you assign it to me? |
@mahdikhashan If you'd like to open a PR, our team will review the work. So feel free to implement. |
Adding additional requirements in the original comment: "Also ensure any other terminal or debug output is accurate in communicating 'compression' or 'upload' as appropriate, as opposed to the terminology of 'processing'. For example, there is a debug log about 'video not found - skipping processing', could this be more explicit to what is being skipped?" |
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
What would you like?
In the context of users being confused around where time is going into 'Processing' videos (is it compression, saving to disk, uploading to cloud?), we want to change the word 'Processing' to 'Compressing' in the terminal output when video's are compressed
Also ensure any other terminal or debug output is accurate in communicating 'compression' or 'upload' as appropriate, as opposed to the terminology of 'processing'. For example, there is a debug log about 'video not found - skipping processing', could this be more explicit to what is being skipped?
Why is this needed?
This is a low effort fix that will allow users to know that changing the videoCompression option directly affects this action Cypress takes that's printed to the terminal.
The text was updated successfully, but these errors were encountered: