Newer
Older
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
# Disallow unused variables (no-unused-vars)
Variables that are declared and not used anywhere in the code are most likely an error due to incomplete refactoring. Such variables take up space in the code and can lead to confusion by readers.
## Rule Details
This rule is aimed at eliminating unused variables, functions, and parameters of functions.
A variable is considered to be used if any of the following are true:
- It represents a function that is called (`doSomething()`)
- It is read (`var y = x`)
- It is passed into a function as an argument (`doSomething(x)`)
- It is read inside of a function that is passed to another function (`doSomething(function() { foo(); })`)
A variable is _not_ considered to be used if it is only ever assigned to (`var x = 5`) or declared.
Examples of **incorrect** code for this rule:
```js
/*eslint no-unused-vars: "error"*/
/*global some_unused_var*/
// It checks variables you have defined as global
some_unused_var = 42;
var x;
// Write-only variables are not considered as used.
var y = 10;
y = 5;
// A read for a modification of itself is not considered as used.
var z = 0;
z = z + 1;
// By default, unused arguments cause warnings.
(function(foo) {
return 5;
})();
// Unused recursive functions also cause warnings.
function fact(n) {
if (n < 2) return 1;
return n * fact(n - 1);
}
// When a function definition destructures an array, unused entries from the array also cause warnings.
function getY([x, y]) {
return y;
}
```
Examples of **correct** code for this rule:
```js
/*eslint no-unused-vars: "error"*/
var x = 10;
alert(x);
// foo is considered used here
myFunc(
function foo() {
// ...
}.bind(this),
);
(function(foo) {
return foo;
})();
var myFunc;
myFunc = setTimeout(function() {
// myFunc is considered used
myFunc();
}, 50);
// Only the second argument from the descructured array is used.
function getY([, y]) {
return y;
}
```
### exported
In environments outside of CommonJS or ECMAScript modules, you may use `var` to create a global variable that may be used by other scripts. You can use the `/* exported variableName */` comment block to indicate that this variable is being exported and therefore should not be considered unused.
Note that `/* exported */` has no effect for any of the following:
- when the environment is `node` or `commonjs`
- when `parserOptions.sourceType` is `module`
- when `ecmaFeatures.globalReturn` is `true`
The line comment `// exported variableName` will not work as `exported` is not line-specific.
Examples of **correct** code for `/* exported variableName */` operation:
```js
/* exported global_var */
var global_var = 42;
```
## Options
This rule takes one argument which can be a string or an object. The string settings are the same as those of the `vars` property (explained below).
By default this rule is enabled with `all` option for variables and `after-used` for arguments.
```CJSON
{
"rules": {
// note you must disable the base rule as it can report incorrect errors
"no-unused-vars": "off",
"@typescript-eslint/no-unused-vars": ["error", {
"vars": "all",
"args": "after-used",
"ignoreRestSiblings": false
}]
}
}
```
### vars
The `vars` option has two settings:
- `all` checks all variables for usage, including those in the global scope. This is the default setting.
- `local` checks only that locally-declared variables are used but will allow global variables to be unused.
#### vars: local
Examples of **correct** code for the `{ "vars": "local" }` option:
```js
/*eslint no-unused-vars: ["error", { "vars": "local" }]*/
/*global some_unused_var */
some_unused_var = 42;
```
### varsIgnorePattern
The `varsIgnorePattern` option specifies exceptions not to check for usage: variables whose names match a regexp pattern. For example, variables whose names contain `ignored` or `Ignored`.
Examples of **correct** code for the `{ "varsIgnorePattern": "[iI]gnored" }` option:
```js
/*eslint no-unused-vars: ["error", { "varsIgnorePattern": "[iI]gnored" }]*/
var firstVarIgnored = 1;
var secondVar = 2;
console.log(secondVar);
```
### args
The `args` option has three settings:
- `after-used` - unused positional arguments that occur before the last used argument will not be checked, but all named arguments and all positional arguments after the last used argument will be checked.
- `all` - all named arguments must be used.
- `none` - do not check arguments.
#### args: after-used
Examples of **incorrect** code for the default `{ "args": "after-used" }` option:
```js
/*eslint no-unused-vars: ["error", { "args": "after-used" }]*/
// 2 errors, for the parameters after the last used parameter (bar)
// "baz" is defined but never used
// "qux" is defined but never used
(function(foo, bar, baz, qux) {
return bar;
})();
```
Examples of **correct** code for the default `{ "args": "after-used" }` option:
```js
/*eslint no-unused-vars: ["error", {"args": "after-used"}]*/
(function(foo, bar, baz, qux) {
return qux;
})();
```
#### args: all
Examples of **incorrect** code for the `{ "args": "all" }` option:
```js
/*eslint no-unused-vars: ["error", { "args": "all" }]*/
// 2 errors
// "foo" is defined but never used
// "baz" is defined but never used
(function(foo, bar, baz) {
return bar;
})();
```
#### args: none
Examples of **correct** code for the `{ "args": "none" }` option:
```js
/*eslint no-unused-vars: ["error", { "args": "none" }]*/
(function(foo, bar, baz) {
return bar;
})();
```
### ignoreRestSiblings
The `ignoreRestSiblings` option is a boolean (default: `false`). Using a [Rest Property](https://github.com/tc39/proposal-object-rest-spread) it is possible to "omit" properties from an object, but by default the sibling properties are marked as "unused". With this option enabled the rest property's siblings are ignored.
Examples of **correct** code for the `{ "ignoreRestSiblings": true }` option:
```js
/*eslint no-unused-vars: ["error", { "ignoreRestSiblings": true }]*/
// 'type' is ignored because it has a rest property sibling.
var { type, ...coords } = data;
```
### argsIgnorePattern
The `argsIgnorePattern` option specifies exceptions not to check for usage: arguments whose names match a regexp pattern. For example, variables whose names begin with an underscore.
Examples of **correct** code for the `{ "argsIgnorePattern": "^_" }` option:
```js
/*eslint no-unused-vars: ["error", { "argsIgnorePattern": "^_" }]*/
function foo(x, _y) {
return x + 1;
}
foo();
```
### caughtErrors
The `caughtErrors` option is used for `catch` block arguments validation.
It has two settings:
- `none` - do not check error objects. This is the default setting.
- `all` - all named arguments must be used.
#### caughtErrors: none
Not specifying this rule is equivalent of assigning it to `none`.
Examples of **correct** code for the `{ "caughtErrors": "none" }` option:
```js
/*eslint no-unused-vars: ["error", { "caughtErrors": "none" }]*/
try {
//...
} catch (err) {
console.error('errors');
}
```
#### caughtErrors: all
Examples of **incorrect** code for the `{ "caughtErrors": "all" }` option:
```js
/*eslint no-unused-vars: ["error", { "caughtErrors": "all" }]*/
// 1 error
// "err" is defined but never used
try {
//...
} catch (err) {
console.error('errors');
}
```
### caughtErrorsIgnorePattern
The `caughtErrorsIgnorePattern` option specifies exceptions not to check for usage: catch arguments whose names match a regexp pattern. For example, variables whose names begin with a string 'ignore'.
Examples of **correct** code for the `{ "caughtErrorsIgnorePattern": "^ignore" }` option:
```js
/*eslint no-unused-vars: ["error", { "caughtErrorsIgnorePattern": "^ignore" }]*/
try {
//...
} catch (ignoreErr) {
console.error('errors');
}
```
## When Not To Use It
If you don't want to be notified about unused variables or function arguments, you can safely turn this rule off.
<sup>Taken with ❤️ [from ESLint core](https://github.com/eslint/eslint/blob/master/docs/rules/no-unused-vars.md)</sup>