-
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
feat: use require to support config via cypress.js #15263
Conversation
Thanks for taking the time to open a PR!
|
Test summaryRun details
View run in Cypress Dashboard ➡️ Flakiness
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
This reverts commit 24e07f6.
We had to revert this for 6.7.1 in #15499 because it caused this issue: #15497. The reason is that with the use of |
This reverts commit 24e07f6.
User facing changelog
cypress.js
instead ofcypress.json
.cypress.js
must confirm to the same API, for example:Additional details
The eventual goal is outlined in this ticket. We need to support runner specific config files; JS is more flexible than JSON for this. The first step to supporting JS as well as JSON for configuration is to load it via
require
instead offs.readJson
.How has the user experience changed?
It is still the same.
PR Tasks
N/A