Tslint: [Feature Request] Add include in linterOptions

Created on 11 Jul 2018  ยท  4Comments  ยท  Source: palantir/tslint

Bug Report

  • __TSLint version__: 5.10.0
  • __TypeScript version__: 2.4.2
  • __Running TSLint via__: npm run lint with pre-commit hook (husky)

TypeScript code being linted

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/**",
        "*"
      ]
    }
  }

Actual behavior

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.

Expected behavior

To have an include option in linterOptions. I think this way, it will be really more simple to adopt progressively tslint.

Thanks guys :)

Accepting PRs Feature Request good first issue ๐ŸŒน R.I.P. ๐ŸŒน

Most helpful comment

I will pick it up

All 4 comments

I agree this would be a good idea! Happy to review PRs by anyone interested in taking this on.

I will pick it up

๐Ÿ’€ _It's time!_ ๐Ÿ’€

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:

  • Recommended: Check if this is available in ESLint + typescript-eslint โœ…
  • _Not Recommended: Fork TSLint locally_ ๐Ÿคทโ€โ™‚๏ธ

๐Ÿ‘‹ 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! ๐Ÿ‘‹

Was this page helpful?
0 / 5 - 0 ratings

Related issues

allbto picture allbto  ยท  3Comments

Ne-Ne picture Ne-Ne  ยท  3Comments

denkomanceski picture denkomanceski  ยท  3Comments

SwintDC picture SwintDC  ยท  3Comments

rajinder-yadav picture rajinder-yadav  ยท  3Comments