-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Use modern ES syntax rather than .call
#29368
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #29368 +/- ##
==========================================
+ Coverage 85.01% 85.10% +0.08%
==========================================
Files 1059 1062 +3
Lines 28277 28352 +75
Branches 4538 4548 +10
==========================================
+ Hits 24040 24129 +89
+ Misses 3074 3061 -13
+ Partials 1163 1162 -1 ☔ View full report in Codecov by Sentry. |
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.
Typo, plus some
and every
are not defined on NodeList
afaik
This pull request has merge conflicts that must be resolved before it can be merged. |
8b9dd92
to
e84b1e7
Compare
This pull request has resolved merge conflicts and is ready for review. |
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.
Looks good now, although I'd like to see it run with the test introduced in #29424 first.
e79926f
to
880830e
Compare
Rebased to include your new test |
I replaced
Object.prototype.hasOwnProperty.call
with the new ES2022Object.hasOwn
.It is not transpiled by Babel in the final files, because this function is supported by all the browsers our BrowsersList is targetting.
If this causes issues, then we can change our supported browsers list and target earlier versions, then it will be polyfilled accordingly.