-
Notifications
You must be signed in to change notification settings - Fork 101
Open
Labels
area: incorrect mergerRule merger with incomplete, incorrect, or invalid rule argumentsRule merger with incomplete, incorrect, or invalid rule argumentsgood 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! 🙏
Description
Overview
@typescript-eslint/ban-types has a fixWith option for types. https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/ban-types.md
Actual Behavior
That fixWith might go away if two rule arguments types lists are merged and the second doesn't have it.
First:
{
types: [
{
fixWith: "number",
typeName: "Number",
},
];
}Second:
{
types: [
{
typeName: "Number",
},
];
}Output:
{
types: [
{
typeName: "Number",
},
];
}Expected Behavior
The output should include fixWith if it exists.
Metadata
Metadata
Assignees
Labels
area: incorrect mergerRule merger with incomplete, incorrect, or invalid rule argumentsRule merger with incomplete, incorrect, or invalid rule argumentsgood 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! 🙏