-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Turn off video recording by default #26157
Comments
Also same will be helpful for other artifacts. I want to have a turn on/turn off option for every artifact: screenshots, videos, report files, etc. |
FYI: this is the main reason we would like this feature. Does an option exist to turn it on dynamically with an flag, that would be nice as well. Using ENV variables? |
ref: https://docs.cypress.io/guides/references/configuration#Environment-Variables |
We have made the decision to have |
The code for this is done, but this has yet to be released. We'll update this issue and reference the changelog when it's released. |
Released in Cypress 13.0.0. |
What would you like?
Cypress has always recorded video by default by default during
cypress run
. Video recording can be turned off by users by updating their configuration file or passing in an environment variable to change config at runtime to setvideo: false
.We'd like to have
video: false
set by default for all users, so that video recording is turned off by default. Any user can setvideo: true
to record video.Why is this needed?
Videos are oftentimes not the best source for debugging Cypress tests that ran in a non-local environment because in some situations videos can be choppy or unavailable (especially in under resourced machines). Videos are also not as useful as the feature to capture and display all logs for debugging that our team is working on. Additionally recording video on lower resourced machines impacts the performance of tests, resulting in slower runtime.
Other
Release and implementation strategy has yet to be determined, but this could be result in a breaking change for existing users.
The text was updated successfully, but these errors were encountered: