fix(jsx): remove deprecated jsx-space-before-closing, close #18

eslint-stylistic · Oct 9, 2023 · c6961ed · c6961ed
1 parent d5e9507
commit c6961ed
Show file tree

Hide file tree

Showing 7 changed files with 0 additions and 312 deletions.
diff --git a/packages/eslint-plugin-jsx/rules.md b/packages/eslint-plugin-jsx/rules.md
@@ -17,6 +17,5 @@
 | [`@stylistic/jsx/jsx-one-expression-per-line`](./rules/jsx-one-expression-per-line) | Require one JSX element per line | ✅ |  |
 | [`@stylistic/jsx/jsx-props-no-multi-spaces`](./rules/jsx-props-no-multi-spaces) | Disallow multiple spaces between inline JSX props | ✅ |  |
 | [`@stylistic/jsx/jsx-sort-props`](./rules/jsx-sort-props) | Enforce props alphabetical sorting | ✅ |  |
-| [`@stylistic/jsx/jsx-space-before-closing`](./rules/jsx-space-before-closing) | Enforce spacing before closing bracket in JSX | ✅ |  |
 | [`@stylistic/jsx/jsx-tag-spacing`](./rules/jsx-tag-spacing) | Enforce whitespace in and around the JSX opening and closing brackets | ✅ |  |
 | [`@stylistic/jsx/jsx-wrap-multilines`](./rules/jsx-wrap-multilines) | Disallow missing parentheses around multiline JSX | ✅ |  |
diff --git a/packages/eslint-plugin-jsx/rules/index.js b/packages/eslint-plugin-jsx/rules/index.js
@@ -14,7 +14,6 @@ module.exports = {
   'jsx-one-expression-per-line': require('./jsx-one-expression-per-line/jsx-one-expression-per-line.js'),
   'jsx-props-no-multi-spaces': require('./jsx-props-no-multi-spaces/jsx-props-no-multi-spaces.js'),
   'jsx-sort-props': require('./jsx-sort-props/jsx-sort-props.js'),
-  'jsx-space-before-closing': require('./jsx-space-before-closing/jsx-space-before-closing.js'),
   'jsx-tag-spacing': require('./jsx-tag-spacing/jsx-tag-spacing.js'),
   'jsx-wrap-multilines': require('./jsx-wrap-multilines/jsx-wrap-multilines.js'),
 }
diff --git a/packages/eslint-plugin-jsx/rules/jsx-space-before-closing/README.md b/packages/eslint-plugin-jsx/rules/jsx-space-before-closing/README.md
diff --git a/packages/eslint-plugin-jsx/rules/jsx-space-before-closing/jsx-space-before-closing.js b/packages/eslint-plugin-jsx/rules/jsx-space-before-closing/jsx-space-before-closing.js
diff --git a/packages/eslint-plugin-jsx/rules/jsx-space-before-closing/jsx-space-before-closing.test.js b/packages/eslint-plugin-jsx/rules/jsx-space-before-closing/jsx-space-before-closing.test.js
diff --git a/packages/metadata/src/metadata.ts b/packages/metadata/src/metadata.ts
@@ -1208,20 +1208,6 @@ export const packages: Readonly<PackageInfo[]> = Object.freeze([
           }
         }
       },
-      {
-        "name": "jsx-space-before-closing",
-        "ruleId": "@stylistic/jsx/jsx-space-before-closing",
-        "originalId": "react/jsx-space-before-closing",
-        "entry": "packages/eslint-plugin-jsx/rules/jsx-space-before-closing/jsx-space-before-closing.js",
-        "docsEntry": "packages/eslint-plugin-jsx/rules/jsx-space-before-closing/README.md",
-        "meta": {
-          "fixable": "code",
-          "docs": {
-            "description": "Enforce spacing before closing bracket in JSX",
-            "recommended": false
-          }
-        }
-      },
       {
         "name": "jsx-tag-spacing",
         "ruleId": "@stylistic/jsx/jsx-tag-spacing",
@@ -2627,19 +2613,6 @@ export const packages: Readonly<PackageInfo[]> = Object.freeze([
           }
         }
       },
-      {
-        "name": "jsx-space-before-closing",
-        "ruleId": "@stylistic/jsx-space-before-closing",
-        "entry": "packages/eslint-plugin-jsx/rules/jsx-space-before-closing/jsx-space-before-closing.js",
-        "docsEntry": "packages/eslint-plugin-jsx/rules/jsx-space-before-closing/README.md",
-        "meta": {
-          "fixable": "code",
-          "docs": {
-            "description": "Enforce spacing before closing bracket in JSX",
-            "recommended": false
-          }
-        }
-      },
       {
         "name": "jsx-tag-spacing",
         "ruleId": "@stylistic/jsx-tag-spacing",

diff --git a/scripts/migrate.ts b/scripts/migrate.ts
@@ -130,7 +130,6 @@ const jsxRules = [
   // 'jsx-props-no-spreading',
   // 'jsx-sort-default-props',
   'jsx-sort-props',
-  'jsx-space-before-closing',
   'jsx-tag-spacing',
   'jsx-wrap-multilines',
 ]