-
Notifications
You must be signed in to change notification settings - Fork 3.4k
feat: Remove Studio as experimental - Turn Studio Beta on for all users of Cypress (no need for config flag) #32571
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
feat: Remove Studio as experimental - Turn Studio Beta on for all users of Cypress (no need for config flag) #32571
Conversation
cypress
|
||||||||||||||||||||||||||||
| Project |
cypress
|
| Branch Review |
mabel/issue-30997-clean-up-experimental-studio
|
| Run status |
|
| Run duration | 14m 25s |
| Commit |
|
| Committer | Jennifer Shehane |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
0
|
|
|
0
|
|
|
0
|
|
|
41
|
| View all changes introduced in this branch ↗︎ | |
Warning
No Report: Something went wrong and we could not generate a report for the Application Quality products.
4322c76 to
1cb608e
Compare
1cb608e to
35f1c7f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mabela416 Just some wording updates.
Manually tested that this works.
|
@mabela416 Docs PR for this change: cypress-io/cypress-documentation#6283 |
jennifer-shehane
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not understanding why all the percy snapshots are off. Seems like a quirk with them?
|
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
Studio: Remove experimentalStudio configuration, make default behavior for all users of E2E tests #30997
Additional details
Remove the experimentalStudio configuration option and turn on Cypress Studio Beta for all users when in end-to-end testing.
Remove the configuration option
Remove any types around the config
Remove any internal use of the configuration in the repo
Remove any tests around the flag
Display a warning message to users when this config option is used - informing them that it's safe to remove this configuration option. See similar warnings in code
Steps to test
experimentalStudio: truein your project'scypress.config.jsthen open your project in the app. In the terminal, you should see a warning about being able to safely remove theexperimentalStudioconfig.How has the user experience changed?
PR Tasks
cypress-documentation?type definitions?Note
Enables Cypress Studio by default for E2E, removes the experimentalStudio config, and updates UI, config, errors, tests, and docs accordingly.
experimentalStudioacross app runner and reporter;studioEnabledand Studio button logic no longer read the flag.e2e.experimentalStudiofrom public options and defaults; add breaking warningEXPERIMENTAL_STUDIO_REMOVEDwith messaging.experimentalStudio; adjust E2E specs to new default behavior.experimentalStudiofrom internalcypress.config.tsfiles (driver/reporter/app).Written by Cursor Bugbot for commit a9fe4fa. This will update automatically on new commits. Configure here.