Skip to content

Commit 5ad7cb0

Browse files
committed
feat!: remove 'no-useless-fragment' rule
1 parent a07e3ad commit 5ad7cb0

File tree

10 files changed

+1
-771
lines changed

10 files changed

+1
-771
lines changed

apps/website/content/docs/roadmap.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ Minimum supported versions:
3737
### Add auto-fix feature to rules that can be auto-fixed safely
3838

3939
- [ ] `function-component-definition`
40-
- [x] `no-useless-fragment`
4140
- [x] `prefer-namespace-import`
4241

4342
### Add suggestion-fix feature to rules that can be fixed interactively
@@ -73,6 +72,7 @@ Minimum supported versions:
7372
- [x] `avoid-shorthand-boolean`
7473
- [x] `avoid-shorthand-fragment`
7574
- [x] `no-comment-textnodes` - Replaced by `jsx-no-comment-textnodes`
75+
- [x] `no-useless-fragment`
7676
- [x] `prefer-react-namespace-import` - Replaced by `prefer-namespace-import`
7777
- [x] `prefer-shorthand-boolean` - Replaced by `jsx-shorthand-boolean`
7878
- [x] `prefer-shorthand-fragment`

apps/website/content/docs/rules/meta.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@
4949
"no-unused-state",
5050
"no-use-context",
5151
"no-useless-forward-ref",
52-
"no-useless-fragment",
5352
"prefer-destructuring-assignment",
5453
"prefer-namespace-import",
5554
"prefer-read-only-props",

apps/website/content/docs/rules/overview.mdx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,6 @@ The `jsx-*` rules check for issues exclusive to JSX syntax, which are absent fro
7878
| [`no-unused-state`](./no-unused-state) | 1️⃣ | | Warns unused class component state | |
7979
| [`no-use-context`](./no-use-context) | 1️⃣ | `🔄` | Replaces usages of `useContext` with `use` | >=19.0.0 |
8080
| [`no-useless-forward-ref`](./no-useless-forward-ref) | 1️⃣ | | Disallow useless `forwardRef` calls on components that don't use `ref`s | |
81-
| [`no-useless-fragment`](./no-useless-fragment) | 0️⃣ | `🔧` `⚙️` | Disallow useless fragment elements | |
8281
| [`prefer-destructuring-assignment`](./prefer-destructuring-assignment) | 0️⃣ | | Enforces destructuring assignment for component props and context | |
8382
| [`prefer-namespace-import`](./prefer-namespace-import) | 0️⃣ | `🔧` | Enforces React is imported via a namespace import | |
8483
| [`prefer-read-only-props`](./prefer-read-only-props) | 0️⃣ | `💭` `🧪` | Enforces read-only props in components | |

packages/plugins/eslint-plugin-react-x/src/configs/recommended.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ export const rules = {
5050
"react-x/no-unused-state": "warn",
5151
"react-x/no-use-context": "warn",
5252
"react-x/no-useless-forward-ref": "warn",
53-
// "react-x/no-useless-fragment": "warn",
5453
// "react-x/prefer-destructuring-assignment": "warn",
5554
// "react-x/prefer-namespace-import": "warn",
5655
// "react-x/prefer-read-only-props": "error",

packages/plugins/eslint-plugin-react-x/src/plugin.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ import noUnusedClassComponentMembers from "./rules/no-unused-class-component-mem
4646
import noUnusedState from "./rules/no-unused-state";
4747
import noUseContext from "./rules/no-use-context";
4848
import noUselessForwardRef from "./rules/no-useless-forward-ref";
49-
import noUselessFragment from "./rules/no-useless-fragment";
5049
import preferDestructuringAssignment from "./rules/prefer-destructuring-assignment";
5150
import preferNamespaceImport from "./rules/prefer-namespace-import";
5251
import preferReadOnlyProps from "./rules/prefer-read-only-props";
@@ -104,7 +103,6 @@ export const plugin = {
104103
"no-unused-state": noUnusedState,
105104
"no-use-context": noUseContext,
106105
"no-useless-forward-ref": noUselessForwardRef,
107-
"no-useless-fragment": noUselessFragment,
108106
"prefer-destructuring-assignment": preferDestructuringAssignment,
109107
"prefer-namespace-import": preferNamespaceImport,
110108
"prefer-read-only-props": preferReadOnlyProps,

packages/plugins/eslint-plugin-react-x/src/rules/no-useless-fragment.md

Lines changed: 0 additions & 157 deletions
This file was deleted.

0 commit comments

Comments
 (0)