use astUtils.getStaticPropertyName in no-prototype-builtins
See original GitHub issueWhat rule do you want to change?
no-prototype-builtins
Does this change cause the rule to produce more or fewer warnings?
more
How will the change be implemented? (New option, new default behavior, etc.)?
new default behavior
Please provide some example code that this change will affect:
/*eslint no-prototype-builtins: "error"*/
var hasBarProperty = foo["hasOwnProperty"]("bar");
var isPrototypeOfBar = foo['isPrototypeOf'](bar);
var barIsEnumerable = foo[`propertyIsEnumerable`]("bar");
What does the rule currently do for this code?
no errors
What will the rule do after it’s changed?
3 errors
Are you willing to submit a pull request to implement this change?
Yes.
Issue Analytics
- State:
- Created 3 years ago
- Reactions:6
- Comments:5 (4 by maintainers)
Top Results From Across the Web
MyDocuments Documentation - Read the Docs
Use built-in function bin() bin(n)[2:] Sting to list: >>> list("Python"). ['P', 'y', 't', 'h', 'o', 'n']. String to tuple: > ...
Read more >unwrapping the module did not yield a `lightningmodule`, got ...
I am using google colab and pytorch lightning to train a classification model.I imported 'LightningModule' but still the error persists. This is the...
Read more >Initial import of gr-zmqblocks OOT p - GNU mailing lists
This is an automated email from the git hooks/post-receive script. jcorgan pushed a commit to branch master in repository gnuradio. commit ...
Read more >Timeline - WebKit Trac
Use set_allow_tcp_listen instead of PORTALLOCATOR_DISABLE_ADAPTER_ENUMERATION. Source/WebKit: When starting to monitor, pass whether to ...
Read more >Renovate Bot Package Diff
+ } // We have not matching html + js (inlined `window. ... any>\n): Promise<void> {\n // global gatsby object to use without...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I’m working on this.
I would argue that this is a bug in the current rule.