-
-
Notifications
You must be signed in to change notification settings - Fork 270
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
Integrate Playwright #1722
Comments
@kylemh still want to do this? |
@kylemh I made a PR for this integration. Should be done with it soon..currently testing it out...hopefully I didn't miss anything. but so far working smoothly.. |
Yes, I want to do this. I also want to try and use https://github.com/kwonoj/swc-plugin-coverage-instrument so that we can finally use SWC (and ditch Istanbul + Babel). This would allow our builds to be nearly twice as fast and our dev server would be MUCH faster. |
@kylemh i'll spend some time on this...and review that repo. |
Im willing to ditch coverage too. So focus on Playwright replacing Cypress first. Coverage can come in a later PR. I'm updating some code in the other open PR too. Trying to get us to Next 14 asap |
alrighty...i'll open up a new PR and work on replacing Cypress with Playwright. |
I want to use Playwright for Automated Visual Testing with Chromatic and eventually ditch snapshot tests + ditch Cypress.
Helpful: https://dev.to/szmydlo/comprehensive-coverage-jestplaywright-in-nextjs-ts-1df2
The text was updated successfully, but these errors were encountered: