-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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 from 16.3.0 to 16.4.0 #22124
Bump jsdom from 16.3.0 to 16.4.0 #22124
Conversation
No bundle size changes comparing 1982dc2...79ff5a7 |
Resolve jsdom/jsdom#3025 first. |
Bumps [jsdom](https://github.com/jsdom/jsdom) from 16.3.0 to 16.4.0. - [Release notes](https://github.com/jsdom/jsdom/releases) - [Changelog](https://github.com/jsdom/jsdom/blob/master/Changelog.md) - [Commits](jsdom/jsdom@16.3.0...16.4.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@@ -1,5 +1,5 @@ | |||
module.exports = { | |||
recursive: true, | |||
reporter: 'dot', | |||
require: [require.resolve('@babel/register'), require.resolve('./test/utils/setup')], | |||
require: [require.resolve('@babel/register'), require.resolve('./test/utils/setupJSDOM')], |
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.
It's really about setting up JSDOM. I was always confused by the two init
and setup
files which are essentially synonyms in this context.
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.
@dependabot squash and merge
function isInJSDOM() { | ||
return /jsdom/.test(window.navigator.userAgent); | ||
} | ||
|
||
function isInKarma() { |
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 usage is specific to karma. Having a isInBrowser = !isInJSDOM
would not be accurate.
131e925
to
863683d
Compare
Also shaving off 10s (50s down from 60s) in |
Bumps jsdom from 16.3.0 to 16.4.0.
Changelog
Sourced from jsdom's changelog.
Commits
5279cfd
Version 16.4.03802e13
Fix several bugs in <img>.completebfa4902
When removing the focused element, focus the body20b3971
Add basic processing for second getComputedStyle() argument4a49111
Fix case insensitivity for translate="" and draggable=""63d24a0
Cache Element's tagNamec2fb8ff
Fix the valueMissing check for <input type=radio>574eb51
Implement focusability for SVG elements252b828
Do not allow focusing disconnected nodesaf7e1d9
Fix form.elements to use form owner instead of only descendantsDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot badge me
will comment on this PR with code to add a "Dependabot enabled" badge to your readmeAdditionally, you can set the following in the
.dependabot/config.yml
file in this repo: