-
Notifications
You must be signed in to change notification settings - Fork 961
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Replace TSLint with ESLint #967
Comments
I have created a draft PR (#969), if you like it I can continue working on it. |
If there are desirable lint passes in ESLint that aren't in TSLint then I say go for it. Lint errors won't block syncing code and they don't break the build so it's ok if they get a little out of sync. |
Also @aomarks already merged ESLint in the Material Web Components project. |
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Ad TSLint is being deprecated in favor of ESLint, I want to propose to replace it.
Also, good moment to add pre-commit hook for linting and format.
References
Proposal
Use
@typescript-eslint
,husky
andlint-staged
.package.json
.eslintrc.json
.eslintignore
The text was updated successfully, but these errors were encountered: