tenthe opened a new issue, #4523:
URL: https://github.com/apache/streampipes/issues/4523
### Body
The UI currently disables this rule in `ui/eslint.config.mjs`:
```js
'@typescript-eslint/no-unused-vars': 'off',
```
We should enable it to catch unused imports, variables, parameters, and
stale code after refactors.
Suggested config:
```js
'@typescript-eslint/no-unused-vars': [
'error',
{
argsIgnorePattern: '^_',
varsIgnorePattern: '^_',
caughtErrorsIgnorePattern: '^_',
destructuredArrayIgnorePattern: '^_',
ignoreRestSiblings: true,
},
],
```
A cleanup is needed first, since enabling it currently creates many lint
errors. Intentionally unused values should be renamed with a leading underscore.
### StreamPipes Committer
I acknowledge that I am a maintainer/committer of the Apache StreamPipes
project.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
