-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add typings #1382
Add typings #1382
Conversation
package.json
Outdated
"no-unused-vars": "off" | ||
"no-unused-vars": "off", | ||
"no-redeclare": "off", | ||
"@typescript-eslint/no-redeclare": ["error"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"@typescript-eslint/no-redeclare": ["error"] | |
"@typescript-eslint/no-redeclare": "error" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What error it prevents in Autoprefixer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no-redeclare
rule forbid to make function overloading. Maybe it is better to move rules to @logux/eslint-config
:
"no-redeclare": "off",
"@typescript-eslint/no-redeclare": "error"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where it could happen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In any place when you will have function overloading or interface declaration merging. no-redeclare
doesn't understand that it is not "redeclare", @typescript-eslint/no-redeclare
handles these cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the suggestion. I released ESLint config 44.0.1 and updated config in Autoprefixer’s master
.
Can you revert config changes and rebase branch (and we are ready for the release)?
Released in 10.2 |
Add typings based on typings from DefinitelyTyped.