lint/correctness/noUnusedVariables: Allow adding ignore pattern

[kubijo]

Our translation system requires having an intl object either in this.props.intl or in scope. This often leads to unused variables, but it would be an error not to have it there… because the generated code would be missing the API it needs.

Usually, this is an easy fix with tools like eslint by adding this variable name to patterns ignored by the lint.

[Conaclos]

You could declare the variable in the javascript.globals setting.

[kubijo]

Hmm, I will try that, but it feels wrong as it is not a global by any measure...

This being the typical case:

function Foo() {
   const intl = useIntl();
   // …
}

[kubijo]

It also does not seem to work…

[Conaclos]

Never mind, I thought about noUndeclaredVariables.

[mvarendorff]

Adding my interest. This is possible in ESLint and I have a use-case for this (auto-loading of files through dynamic import which are then registered into a DI-like structure) where I would like to allow exceptions for certain class suffixes. I am down to make a PR if greenlit!

[dyc3]

Sure, go for it. Shouldn't be too hard to add an ignore option. Make the PR against the next branch.