-
Notifications
You must be signed in to change notification settings - Fork 13
chore: re-enable playwright parallelization #551
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
Conversation
We had turned this off in a blind attempt to fix flakiness. We recently *actually* fixed flakiness, which was unrelated to this.
✅ Deploy Preview for remix-serverless ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for remix-edge ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| testDir: './tests/e2e', | ||
| /* Run tests in files in parallel */ | ||
| fullyParallel: true, | ||
| workers: 1, |
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.
(the default is dynamic, based on number of cpu cores)
pieh
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.
🚤
pieh
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.
Description
We had turned this off long ago in a blind attempt to fix flakiness.
We recently actually fixed flakiness, which turned out to be unrelated to this.
This significantly speeds up our CI, from ~12m to ~6m.