-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improve README with TSLint migration info #3
Conversation
@@ -8,8 +8,48 @@ This package provide the following ESLint custom rules for Typescript projects. | |||
|
|||
This repository replace [italia-tslint-rules](https://github.com/pagopa/io-tslint-rules) after TSLint deprecation. | |||
|
|||
The following TSLint rules (included inside `italia-tslint-rules`) are not supported for eslint at the moment and are missing in this package: | |||
``` | |||
bool-param-default |
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 does this list come from?
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.
This has been the list that I've produced working on #1.
The steps that I've followed was:
- Follow TSLint to ESLint migration guide on the project
io-backend
- Running
npx tslint-to-eslint-config
that produces a list of TSLint rules not available for ESLint (77 rules) - Manually check each one for an alternative ESLint rules/plugins
- Verify each alternative ESLint rule/plugin on a testing file
The remaining was this list.
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.
great! can you add this comment to the readme?
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.
Fixed in b5a9a7c
List of Changes
Add migration section inside README
Motivation and Context
To give more detailed information above migration from TSLint and
italia-tslint-rules
to ESLint a new section was added with this PR inside the README.The README now includes the list of rules present into
italia-tslint-rules
and missing inside@pagopa/eslint-config
.How Has This Been Tested?
No tests needed.
Types of changes
Checklist:
Question:
Can we enable
eslint
even on.test.ts
files? So we could have the same code style restrictions on tests.