-
Notifications
You must be signed in to change notification settings - Fork 466
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
Bump JSDOM to 21.x #1206
base: main
Are you sure you want to change the base?
Bump JSDOM to 21.x #1206
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit be5a252:
|
Codecov Report
@@ Coverage Diff @@
## main #1206 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 24 24
Lines 998 998
Branches 327 327
=========================================
Hits 998 998
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
"devDependencies": { | ||
"@testing-library/jest-dom": "^5.11.6", | ||
"@types/lz-string": "^1.3.34", | ||
"jest-in-case": "^1.0.2", | ||
"jest-snapshot-serializer-ansi": "^1.0.0", | ||
"jest-watch-select-projects": "^2.0.0", | ||
"jsdom": "^16.4.0", | ||
"jsdom": "^21.1.0", |
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.
quite the hefty bump pulling in unrelated breaking changes. Will bump kcd-scripts to jest 29 first, then update kcd-scripts here
BREAKING CHANGE: Minimum supported Node.js version is now 14.x
* Bump kcd-scripts to 9.x * Fix lint issues * fix lint
* Bump kcd-scripts to 13.0.0 * Fix Jest warnings
e991be4
to
be5a252
Compare
Testing JSDOM 21.x. Only meant as a proof-of-concept for bumping it in
jest-environment-jsdom