-
Notifications
You must be signed in to change notification settings - Fork 101
Open
Labels
good first issueGood for newcomers, please hop on! 🙌Good for newcomers, please hop on! 🙌status: accepting prsPlease, send a pull request to resolve this! 🙏Please, send a pull request 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, please hop on! 🙌Good for newcomers, please hop on! 🙌status: accepting prsPlease, send a pull request to resolve this! 🙏Please, send a pull request to resolve this! 🙏type: cleanupCode smells, incorrect tests, build systems, or other internal shenanigansCode smells, incorrect tests, build systems, or other internal shenanigans