-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[infra] Use playwright
docker image
#18186
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
Deploy preview: https://deploy-preview-18186--material-ui-x.netlify.app/ Bundle size reportTotal Size Change: 0B(0.00%) - Total Gzip Change: 0B(0.00%) |
Need to remove the entire |
We might actually do what we did before which was to link the playwright one globally, as our issue is downloading from the Ubuntu archive, which this would still be doing 😅 |
Yes, but this does benefit from the CircleCI caching and often uses the existing cache instead of pulling a fresh container. 🤔 |
Argos changes mostly due to system changes, there are some changes related to print dialog, but I find them acceptable. |
Agreed. |
Revert changes introduced in #17214, since we no longer use mocha and tests work on vitest with Node 22.
Allowed by #18071.
This should hopefully avoid the insanely flaky CI lately. 🙈