sindresorhus/eslint-plugin-unicorn
Rule proposal: `no-unused-return-value`
Geschlossen
#741 geöffnet am 22.05.2020
help wantednew rule
Repository-Metriken
- Stars
- (5.022 Sterne)
- PR-Merge-Metriken
- (Durchschn. Merge 4h 30m) (26 gemergte PRs in 30 T)
Beschreibung
Details here: https://github.com/eslint/eslint/issues/12980
The idea is that many builtin functions return values and if those return values are not used, the code is essentially a noop and can be removed.
The discussion got stale because it seems hard for a plugin to correctly judge the type of a variable which is essential for a rule like this so it may require Typescript to really be useful. Also there was the problem of getters which would trigger false positives but I generally see getters as a obscure feature that should not be used.
Fail
const arr = [{name: 'eslint'}];
arr.map(item => item.name); // error: Unused Array.prototype.map return value
console.log(arr);
Pass
const arr = [{name: 'eslint'}];
console.log(arr);
Fail
let i = 0;
const arr = [1,2,3];
arr.map((item) => { // error: Unused Array.prototype.map return value
i += 1;
});
Pass
let i = 0;
const arr = [1,2,3];
arr.forEach((item) => {
i += 1;
});