-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
fix: add task tests iteratively #5235
Conversation
✅ Deploy Preview for fastidious-cascaron-4ded94 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I'll fix the checks and add the changeset soon. Slow train internet won't let me install the dependencies. |
|
Maybe this one too? (though UI might not be usable if there are that much of test cases...)
(btw, chrome driver issue will be fixed in #5240) |
Good call! I've changed that part as well, just to be safe. |
7414afc
to
6240415
Compare
The spread operator may result in an exceeded stack size limit for a very large number of tests.
Closes #5234. The spread operator may result in an exceeded stack size limit for a very large number of tests.
Description
See #5234, a very large number of tests results in an exceeded stack size limit due to the usage of the spread operator.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
pnpm-lock.yaml
unless you introduce a new test example.Tests
pnpm test:ci
.Documentation
pnpm run docs
command.Changesets
feat:
,fix:
,perf:
,docs:
, orchore:
.