npm run lint with pre-commit hook (husky)All our app code is being linted.
with tslint.json configuration:
{
"rulesDirectory": [
"node_modules/codelyzer"
],
"rules": {
// Lot of stuff here...
},
"linterOptions": {
"exclude": [
"node_modules/**",
"src/app/admin/**",
"src/app/core/**",
"src/app/custom/**",
"src/app/feedbacks/**",
"src/app/formats/**",
"src/app/groups/**",
"src/app/login/**",
"src/app/main/**",
"src/app/meets/**",
"src/app/profile/**",
"src/app/server/**",
"src/app/shared/**",
"src/app/store/**",
"src/app/widget/**",
"src/app/**",
"src/**",
"test/**",
"*"
]
}
}
Well, my team and I want to gradually adopt tslint, to have a consistant coding style.
However, the only way I found is to exclude all our app folders in linter options. And this is pretty messy.
I have looked for another better suitable option, but found none.
To have an include option in linterOptions. I think this way, it will be really more simple to adopt progressively tslint.
Thanks guys :)
I agree this would be a good idea! Happy to review PRs by anyone interested in taking this on.
I will pick it up
TSLint is being deprecated and no longer accepting pull requests for major new changes or features. See #4534. ๐ฑ
If you'd like to see this change implemented, you have two choices:
๐ It was a pleasure open sourcing with you!
_If you believe this message was posted here in error, please comment so we can re-open the issue!_
๐ค Beep boop! ๐ TSLint is deprecated ๐ _(#4534)_ and you should switch to typescript-eslint! ๐ค
๐ This issue is being locked to prevent further unnecessary discussions. Thank you! ๐
Most helpful comment
I will pick it up