Consider all arguments used if `arguments` is used.
See original GitHub issueWhat rule do you want to change?
no-unused-vars
Does this change cause the rule to produce more or fewer warnings?
fewer
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:
function buildARNString(partition, service, region, accountID, resourceInfo) {
return ARN.ify(["arn", ...arguments].join(':')).toString();
}
What does the rule currently do for this code?
Warns about unused arguments (all of them).
What will the rule do after it’s changed?
No warnings about unused arguments, because they are actually used via arguments
.
Are you willing to submit a pull request to implement this change?
Not for now, zero experience with ESLint rule development.
Issue Analytics
- State:
- Created 3 years ago
- Reactions:1
- Comments:14 (9 by maintainers)
Top Results From Across the Web
The Argument
An argument is a line of reasoning designed to prove a point. Arguments can be simple, expressed in a few lines, or very...
Read more >Argument: The Basics | Department of Communication
Argument is not simply restating the same claims and reasons, rather it is supporting, modifying or defending positions accordingly.
Read more >How to Evaluate Arguments - FutureLearn
When evaluating arguments, we have two main questions to ask: Do the premises provide enough logical support for the conclusion? Are the premises...
Read more >Introduction to arguments (article) | Khan Academy
An argument is a set of statements made up, at minimum, of the following parts: A main conclusion: This statement is a claim...
Read more >Deductive and Inductive Arguments
All arguments are made better by having true premises, of course, ... In response, it might be advised to look for the use...
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 Free
Top 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
This case is already supported with the
{ "args": "after-used" }
option:online demo
There is also
{ "args": "none" }
to completely ignore params:online demo
Thanks for clarifying this!
For typescript code, you should use
@typescript-eslint/no-unused-vars
or@typescript-eslint/no-unused-vars-experimental
, instead of the coreno-unused-vars
rule which isn’t aware of ts specifics.@typescript-eslint/eslint-plugin
extends some core rules to provide support for ts code.