-
Notifications
You must be signed in to change notification settings - Fork 100
Open
Labels
good first issueGood for newcomers; welcome aboard!Good for newcomers; welcome aboard!status: accepting prsPlease, send in a PR to resolve this! β¨Please, send in a PR to resolve this! β¨type: cleanupCode smells, incorrect tests, build systems, or other internal shenanigansCode smells, incorrect tests, build systems, or other internal shenanigans
Description
π Feature Request
Existing Behavior
We do have declared a merger for the @typescript-eslint/no-unnecessary-type-assertion
rule but this doesn't seems to require it as it is pretty much a 1:1 converter with the whitelist of types to ignore being mapped into typesToIgnore
as stated in the ESLint docs.
- TSLint Rule:
no-unnecessary-type-assertion
- ESLint Rule:
@typescript-eslint/no-unnecessary-type-assertion
Change Proposal
Cleanup the rule by removing the merger and adding the logic into the converter.
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomers; welcome aboard!Good for newcomers; welcome aboard!status: accepting prsPlease, send in a PR to resolve this! β¨Please, send in a PR to resolve this! β¨type: cleanupCode smells, incorrect tests, build systems, or other internal shenanigansCode smells, incorrect tests, build systems, or other internal shenanigans