Skip to content
#

tslint

Here are 689 public repositories matching this topic...

Zamiell
Zamiell commented Apr 27, 2022

I am doing a PR for a new linter rule.
I would like to test this new linter rule on an existing repository, but it is unclear how to do that.
Normally, I would use npm link for this purpose, but that doesn't seem to be working properly, presumably because this is a monorepo.

Please enumerate the steps for testing a PR in [CONTRIBUTING.md](https://github.com/typescript-eslint/typescript-esli

good first issue documentation accepting prs
sophia-belous
sophia-belous commented Apr 6, 2022

PascalCase and snake_case configuration options never get migrated to eslint because formats variable is not used anywhere.
See: https://github.com/typescript-eslint/tslint-to-eslint-config/blob/7c92c875a43018c7f5011283cf14da2a173a698f/src/converters/lintConfigs/rules/ruleConverters/variable-name.ts#L40-L63

type: bug status: accepting prs good first issue area: incorrect converter
saadq
saadq commented Jul 17, 2018

TSLint version >= 2.9 seems to have deprecated no-unused-variable.

...the changes in 2.9 make it much more complex for this rule to work as intended, especially with respect to the ignorePattern option.

It looks like people should now just make sure that their tsconfig.json has the noUnusedLocals option enabled which covers most of what

good first issue tslint

Improve this page

Add a description, image, and links to the tslint topic page so that developers can more easily learn about it.

Curate this topic

Add this topic to your repo

To associate your repository with the tslint topic, visit your repo's landing page and select "manage topics."

Learn more