Don't bother with a rule merger if rule arguments are the same #1086
Labels
good first issue
Good for newcomers; welcome aboard!
status: accepting prs
Please, send in a PR to resolve this! ✨
type: enhancement
New feature or request
Overview
As noted in #1085: some community rulesets have overlapping rules to others with the same output ESLint rules: e.g. both
convertAdjacentOverloadSignatures
andconvertConsecutiveOverloads
outputting@typescript-eslint/adjacent-overload-signatures
.Actual Behavior
A rule merger will be requested the second+ time an ESLint rule is seen in a converter's output: even if it outputs the same rule arguments as the ESLint rule's previously output rule arguments.
Expected Behavior
A rule merger should only be called for if an ESLint's rules are seen a second+ time with new rule arguments.
The text was updated successfully, but these errors were encountered: