Remove deprecated rules (#652)

ota-meshi · Oct 7, 2023 · 04a6f52 · 04a6f52
1 parent f98f7ab
commit 04a6f52
Show file tree

Hide file tree

Showing 22 changed files with 139 additions and 532 deletions.
diff --git a/.changeset/fifty-schools-run.md b/.changeset/fifty-schools-run.md
@@ -0,0 +1,5 @@
+---
+"eslint-plugin-regexp": mmajor
+---
+
+Remove deprecated rules
diff --git a/.eslint-doc-generatorrc.js b/.eslint-doc-generatorrc.js
@@ -37,10 +37,14 @@ const config = {
           ([, rule]) => rule.meta.docs.category === 'Stylistic Issues' && !rule.meta.deprecated
         ),
       },
-      {
-        title: 'Deprecated',
-        rules: rules.filter(([, rule]) => rule.meta.deprecated),
-      },
+      ...rules.some(([, rule]) => rule.meta.deprecated)
+        ? [
+          {
+            title: 'Deprecated',
+            rules: rules.filter(([, rule]) => rule.meta.deprecated),
+          }
+        ]
+        : [],
     ];
   },
   urlRuleDoc(name, page) {

diff --git a/README.md b/README.md
@@ -202,17 +202,23 @@ The `plugin:regexp/all` config enables all rules. It's meant for testing, not fo
 | [sort-flags](https://ota-meshi.github.io/eslint-plugin-regexp/rules/sort-flags.html)                                             | require regex flags to be sorted                                       | ✅  |    | 🔧 |    |
 | [unicode-escape](https://ota-meshi.github.io/eslint-plugin-regexp/rules/unicode-escape.html)                                     | enforce consistent usage of unicode escape or unicode codepoint escape |    |    | 🔧 |    |
 
-### Deprecated
+<!-- end auto-generated rules list -->
 
-| Name                                                                                                                       | Description                                   | 💼 | ⚠️ | 🔧 | 💡 |
-| :------------------------------------------------------------------------------------------------------------------------- | :-------------------------------------------- | :- | :- | :- | :- |
-| [no-assertion-capturing-group](https://ota-meshi.github.io/eslint-plugin-regexp/rules/no-assertion-capturing-group.html)   | disallow capturing group that captures empty. |    |    |    |    |
-| [no-useless-exactly-quantifier](https://ota-meshi.github.io/eslint-plugin-regexp/rules/no-useless-exactly-quantifier.html) | disallow unnecessary exactly quantifier       |    |    |    |    |
-| [no-useless-non-greedy](https://ota-meshi.github.io/eslint-plugin-regexp/rules/no-useless-non-greedy.html)                 | disallow unnecessarily non-greedy quantifiers |    |    | 🔧 |    |
-| [order-in-character-class](https://ota-meshi.github.io/eslint-plugin-regexp/rules/order-in-character-class.html)           | enforces elements order in character class    |    |    | 🔧 |    |
-| [prefer-t](https://ota-meshi.github.io/eslint-plugin-regexp/rules/prefer-t.html)                                           | enforce using `\t`                            |    |    | 🔧 |    |
+<!--REMOVED_RULES_START-->
 
-<!-- end auto-generated rules list -->
+### Removed
+
+- :no_entry: These rules have been removed in a previous major release, after they have been deprecated for a while.
+
+| Rule ID | Replaced by | Removed in version |
+|:--------|:------------|:-------------------|
+| [no-assertion-capturing-group](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/no-assertion-capturing-group.md) | [regexp/no-empty-capturing-group](./no-empty-capturing-group.md) | v2.0.0 |
+| [no-useless-exactly-quantifier](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/no-useless-exactly-quantifier.md) | [regexp/no-useless-quantifier](./no-useless-quantifier.md), [regexp/no-zero-quantifier](./no-zero-quantifier.md) | v2.0.0 |
+| [no-useless-non-greedy](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/no-useless-non-greedy.md) | [regexp/no-useless-lazy](./no-useless-lazy.md) | v2.0.0 |
+| [order-in-character-class](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/order-in-character-class.md) | [regexp/sort-character-class-elements](./sort-character-class-elements.md) | v2.0.0 |
+| [prefer-t](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/prefer-t.md) | [regexp/control-character-escape](./control-character-escape.md) | v2.0.0 |
+
+<!--REMOVED_RULES_END-->
 
 ## :gear: Settings
 

diff --git a/docs/rules/index.md b/docs/rules/index.md
@@ -108,14 +108,20 @@ sidebarDepth: 0
 | [sort-flags](sort-flags.md)                                             | require regex flags to be sorted                                       | ✅  |    | 🔧 |    |
 | [unicode-escape](unicode-escape.md)                                     | enforce consistent usage of unicode escape or unicode codepoint escape |    |    | 🔧 |    |
 
-## Deprecated
+<!-- end auto-generated rules list -->
 
-| Name                                                              | Description                                   | 💼 | ⚠️ | 🔧 | 💡 |
-| :---------------------------------------------------------------- | :-------------------------------------------- | :- | :- | :- | :- |
-| [no-assertion-capturing-group](no-assertion-capturing-group.md)   | disallow capturing group that captures empty. |    |    |    |    |
-| [no-useless-exactly-quantifier](no-useless-exactly-quantifier.md) | disallow unnecessary exactly quantifier       |    |    |    |    |
-| [no-useless-non-greedy](no-useless-non-greedy.md)                 | disallow unnecessarily non-greedy quantifiers |    |    | 🔧 |    |
-| [order-in-character-class](order-in-character-class.md)           | enforces elements order in character class    |    |    | 🔧 |    |
-| [prefer-t](prefer-t.md)                                           | enforce using `\t`                            |    |    | 🔧 |    |
+<!--REMOVED_RULES_START-->
 
-<!-- end auto-generated rules list -->
+## Removed
+
+- :no_entry: These rules have been removed in a previous major release, after they have been deprecated for a while.
+
+| Rule ID | Replaced by | Removed in version |
+|:--------|:------------|:-------------------|
+| [no-assertion-capturing-group](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/no-assertion-capturing-group.md) | [regexp/no-empty-capturing-group](./no-empty-capturing-group.md) | v2.0.0 |
+| [no-useless-exactly-quantifier](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/no-useless-exactly-quantifier.md) | [regexp/no-useless-quantifier](./no-useless-quantifier.md), [regexp/no-zero-quantifier](./no-zero-quantifier.md) | v2.0.0 |
+| [no-useless-non-greedy](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/no-useless-non-greedy.md) | [regexp/no-useless-lazy](./no-useless-lazy.md) | v2.0.0 |
+| [order-in-character-class](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/order-in-character-class.md) | [regexp/sort-character-class-elements](./sort-character-class-elements.md) | v2.0.0 |
+| [prefer-t](https://github.com/ota-meshi/eslint-plugin-regexp/blob/v1.15.0/docs/rules/prefer-t.md) | [regexp/control-character-escape](./control-character-escape.md) | v2.0.0 |
+
+<!--REMOVED_RULES_END-->
diff --git a/docs/rules/no-assertion-capturing-group.md b/docs/rules/no-assertion-capturing-group.md
diff --git a/docs/rules/no-useless-exactly-quantifier.md b/docs/rules/no-useless-exactly-quantifier.md
diff --git a/docs/rules/no-useless-non-greedy.md b/docs/rules/no-useless-non-greedy.md
diff --git a/docs/rules/order-in-character-class.md b/docs/rules/order-in-character-class.md
diff --git a/docs/rules/prefer-t.md b/docs/rules/prefer-t.md
diff --git a/lib/rules/no-assertion-capturing-group.ts b/lib/rules/no-assertion-capturing-group.ts
diff --git a/lib/rules/no-useless-exactly-quantifier.ts b/lib/rules/no-useless-exactly-quantifier.ts
diff --git a/lib/rules/no-useless-non-greedy.ts b/lib/rules/no-useless-non-greedy.ts
diff --git a/lib/rules/order-in-character-class.ts b/lib/rules/order-in-character-class.ts