-
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
base: develop
Are you sure you want to change the base?
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 | 20m 21s |
Commit |
|
Committer | Jennifer Shehane |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
12
|
|
1102
|
|
0
|
|
26662
|
View all changes introduced in this branch ↗︎ |
UI Coverage
45.01%
|
|
---|---|
|
188
|
|
158
|
Accessibility
97.71%
|
|
---|---|
|
4 critical
8 serious
2 moderate
2 minor
|
|
110
|
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.
The ${fmt.highlight(`experimentalStudio`)} experiment is currently only supported for End to End Testing. | ||
If you have feedback about the experiment, please join the discussion here: http://on.cypress.io/studio-beta` | ||
The ${fmt.highlight(`experimentalStudio`)} was removed in ${fmt.cypressVersion(`15.4.0`)}. |
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.
I think I would add that Studio is on by default here. We removed the feature at one point and there was a lot of backlash, so I don't want users to think it's gone as a feature.
The ${fmt.highlight(`experimentalStudio`)} was removed in ${fmt.cypressVersion(`15.4.0`)}. | |
The ${fmt.highlight(`experimentalStudio`)} option was removed in ${fmt.cypressVersion(`15.4.0`)}. | |
Cypress Studio is now available for all users. |
@mabela416 Docs PR for this change: cypress-io/cypress-documentation#6283 |
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: true
in your project'scypress.config.js
then open your project in the app. In the terminal, you should see a warning about being able to safely remove theexperimentalStudio
config.How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?