sindresorhus/eslint-plugin-unicorn
Rule proposal: `no-return-prototype-method`
Chiusa
#1098 aperta il 11 feb 2021
help wantednew rule
Metriche repository
- Star
- (5022 stelle)
- Metriche merge PR
- (Merge medio 4h 30m) (26 PR mergiate in 30 g)
Descrizione
It's easy to forget actually calling a prototype method which leads to possibly hard-to-debug errors.
Since e.g. many array methods accept a callback or only an optional parameter, this could be handled with an allowlist per global object, e.g. Array.prototype.map wouldnt need to be checked for.
Fail
function foo(myString) {
return string.slice(3).toLowerCase
}
function foo(myArr) {
return myArr.map(val => val ** 2).reverse
}
Pass
function foo(myString) {
return string.slice(3).toLowerCase()
}
function foo(myArr) {
return myArr.map(val => val ** 2).reverse()
}