Skip to content

Commit 6e366a4

Browse files
author
Josh Goldberg
authored
Added jsx-space-before-trailing-slash converter (#861)
* Added jsx-space-before-trailing-slash converter * Updated to non deprecated rule * Respecting default options
1 parent 98a9d58 commit 6e366a4

File tree

3 files changed

+44
-0
lines changed

3 files changed

+44
-0
lines changed

src/converters/lintConfigs/rules/ruleConverters.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,7 @@ import { convertJsxBooleanValue } from "./ruleConverters/eslint-plugin-react/jsx
181181
import { convertJsxCurlySpacing } from "./ruleConverters/eslint-plugin-react/jsx-curly-spacing";
182182
import { convertJsxEqualsSpacing } from "./ruleConverters/eslint-plugin-react/jsx-equals-spacing";
183183
import { convertJsxKey } from "./ruleConverters/eslint-plugin-react/jsx-key";
184+
import { convertJsxSpaceBeforeTrailingSlash } from "./ruleConverters/eslint-plugin-react/jsx-space-before-trailing-slash";
184185
import { convertJsxNoBind } from "./ruleConverters/eslint-plugin-react/jsx-no-bind";
185186
import { convertJsxNoLambda } from "./ruleConverters/eslint-plugin-react/jsx-no-lambda";
186187
import { convertJsxSelfClose } from "./ruleConverters/eslint-plugin-react/jsx-self-close";
@@ -248,6 +249,7 @@ export const ruleConverters = new Map([
248249
["jsx-no-bind", convertJsxNoBind],
249250
["jsx-no-lambda", convertJsxNoLambda],
250251
["jsx-self-close", convertJsxSelfClose],
252+
["jsx-space-before-trailing-slash", convertJsxSpaceBeforeTrailingSlash],
251253
["jsx-wrap-multiline", convertJsxWrapMultiline],
252254
["label-position", convertLabelPosition],
253255
["linebreak-style", convertLinebreakStyle],
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
import { RuleConverter } from "../../ruleConverter";
2+
3+
export const convertJsxSpaceBeforeTrailingSlash: RuleConverter = () => {
4+
return {
5+
rules: [
6+
{
7+
ruleArguments: [
8+
{
9+
afterOpening: "allow",
10+
closingSlash: "allow",
11+
},
12+
],
13+
ruleName: "react/jsx-tag-spacing",
14+
},
15+
],
16+
plugins: ["eslint-plugin-react"],
17+
};
18+
};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
import { convertJsxSpaceBeforeTrailingSlash } from "../jsx-space-before-trailing-slash";
2+
3+
describe(convertJsxSpaceBeforeTrailingSlash, () => {
4+
test("conversion without arguments", () => {
5+
const result = convertJsxSpaceBeforeTrailingSlash({
6+
ruleArguments: [],
7+
});
8+
9+
expect(result).toEqual({
10+
rules: [
11+
{
12+
ruleArguments: [
13+
{
14+
afterOpening: "allow",
15+
closingSlash: "allow",
16+
},
17+
],
18+
ruleName: "react/jsx-tag-spacing",
19+
},
20+
],
21+
plugins: ["eslint-plugin-react"],
22+
});
23+
});
24+
});

0 commit comments

Comments
 (0)