Skip to content

Commit 047138d

Browse files
committed
refactor!: remove no-complex-conditional-rendering rule and associated tests
1 parent f798e1f commit 047138d

File tree

8 files changed

+4
-763
lines changed

8 files changed

+4
-763
lines changed

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ The `jsx-*` rules check for issues exclusive to JSX syntax, which are absent fro
3434
| [`jsx-no-comment-textnodes`](./jsx-no-comment-textnodes) | 1️⃣ | | Prevents comments from being inserted as text nodes | |
3535
| [`jsx-no-duplicate-props`](./jsx-no-duplicate-props) | 1️⃣ | | Disallow duplicate props in JSX elements | |
3636
| [`jsx-no-undef`](./jsx-no-undef) | 0️⃣ | | Disallow undefined variables in JSX elements | |
37-
| [`jsx-shorthand-boolean`](./jsx-shorthand-boolean) | 0️⃣ | | Enforces the use of shorthand or explicit boolean attributes | |
37+
| [`jsx-shorthand-boolean`](./jsx-shorthand-boolean) | 0️⃣ | | Enforces the use of shorthand or explicit boolean attributes | |
3838
| [`jsx-shorthand-fragment`](./jsx-shorthand-fragment) | 0️⃣ | | Enforces the use of shorthand `<>` or `</>` syntax or `<React.Fragment>` element | |
3939
| [`jsx-uses-react`](./jsx-uses-react) | 1️⃣ | | Marks React variables as used when JSX is used | |
4040
| [`jsx-uses-vars`](./jsx-uses-vars) | 1️⃣ | | Marks variables used in JSX elements as used | |
@@ -48,7 +48,6 @@ The `jsx-*` rules check for issues exclusive to JSX syntax, which are absent fro
4848
| [`no-children-to-array`](./no-children-to-array) | 1️⃣ | | Disallow `Children.toArray` | |
4949
| [`no-class-component`](./no-class-component) | 0️⃣ | | Disallow class components except for error boundaries | |
5050
| [`no-clone-element`](./no-clone-element) | 1️⃣ | | Disallow `cloneElement` | |
51-
| [`no-complex-conditional-rendering`](./no-complex-conditional-rendering) | 0️⃣ | `🧪` | Disallow complex conditional rendering in JSX expressions | |
5251
| [`no-component-will-mount`](./no-component-will-mount) | 2️⃣ | `🔄` | Replaces usages of `componentWillMount` with `UNSAFE_componentWillMount` | >=16.3.0 |
5352
| [`no-component-will-receive-props`](./no-component-will-receive-props) | 2️⃣ | `🔄` | Replaces usages of `componentWillReceiveProps` with `UNSAFE_componentWillReceiveProps` | >=16.3.0 |
5453
| [`no-component-will-update`](./no-component-will-update) | 2️⃣ | `🔄` | Replaces usages of `componentWillUpdate` with `UNSAFE_componentWillUpdate` | >=16.3.0 |

apps/website/migration/index.ts

Lines changed: 0 additions & 67 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,5 @@
11
/* eslint-disable perfectionist/sort-objects */
22
export const redirects = [
3-
// Redirects for old documentation links
4-
{
5-
source: "/docs/getting-started/javascript-with-alternative-parser",
6-
destination: "/docs/using-an-alternative-parser/babel-eslint-parser",
7-
permanent: true,
8-
},
9-
{
10-
source: "/docs/getting-started/typescript-with-alternative-parser",
11-
destination: "/docs/using-an-alternative-parser/ts-blank-eslint-parser",
12-
permanent: true,
13-
},
143
// Redirects for old preset links
154
{
165
source: "/docs/rules/overview#core-rules",
@@ -22,60 +11,4 @@ export const redirects = [
2211
destination: "/docs/rules/overview#x-rules",
2312
permanent: true,
2413
},
25-
// Redirects for old rule names
26-
{
27-
source: "/docs/rules/use-jsx-vars",
28-
destination: "/docs/rules/jsx-uses-vars",
29-
permanent: true,
30-
},
31-
{
32-
source: "/docs/rules/no-duplicate-jsx-props",
33-
destination: "/docs/rules/jsx-no-duplicate-props",
34-
permanent: true,
35-
},
36-
{
37-
source: "/docs/rules/no-complicated-conditional-rendering",
38-
destination: "/docs/rules/no-complex-conditional-rendering",
39-
permanent: true,
40-
},
41-
{
42-
source: "/docs/rules/ensure-forward-ref-using-ref",
43-
destination: "/docs/rules/no-useless-forward-ref",
44-
permanent: true,
45-
},
46-
{
47-
source: "/docs/rules/no-nested-components",
48-
destination: "/docs/rules/no-nested-component-definitions",
49-
permanent: true,
50-
},
51-
{
52-
source: "/docs/rules/dom-no-children-in-void-dom-elements",
53-
destination: "/docs/rules/dom-no-void-elements-with-children",
54-
permanent: true,
55-
},
56-
{
57-
source: "/docs/rules/hooks-extra-ensure-use-memo-has-non-empty-deps",
58-
destination: "/docs/rules/hooks-extra-no-unnecessary-use-memo",
59-
permanent: true,
60-
},
61-
{
62-
source: "/docs/rules/hooks-extra-ensure-use-callback-has-non-empty-deps",
63-
destination: "/docs/rules/hooks-extra-no-unnecessary-use-callback",
64-
permanent: true,
65-
},
66-
{
67-
source: "/docs/rules/hooks-extra-ensure-custom-hooks-using-other-hooks",
68-
destination: "/docs/rules/hooks-extra-no-unnecessary-use-prefix",
69-
permanent: true,
70-
},
71-
{
72-
source: "/docs/rules/hooks-extra-no-redundant-custom-hook",
73-
destination: "/docs/rules/hooks-extra-no-unnecessary-use-prefix",
74-
permanent: true,
75-
},
76-
{
77-
source: "/docs/rules/hooks-extra-no-useless-custom-hooks",
78-
destination: "/docs/rules/hooks-extra-no-unnecessary-use-prefix",
79-
permanent: true,
80-
},
8114
] as const;

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ import noChildrenProp from "./rules/no-children-prop";
1616
import noChildrenToArray from "./rules/no-children-to-array";
1717
import noClassComponent from "./rules/no-class-component";
1818
import noCloneElement from "./rules/no-clone-element";
19-
import noComplexConditionalRendering from "./rules/no-complex-conditional-rendering";
2019
import noComponentWillMount from "./rules/no-component-will-mount";
2120
import noComponentWillReceiveProps from "./rules/no-component-will-receive-props";
2221
import noComponentWillUpdate from "./rules/no-component-will-update";
@@ -77,7 +76,6 @@ export const plugin = {
7776
"no-children-to-array": noChildrenToArray,
7877
"no-class-component": noClassComponent,
7978
"no-clone-element": noCloneElement,
80-
"no-complex-conditional-rendering": noComplexConditionalRendering,
8179
"no-component-will-mount": noComponentWillMount,
8280
"no-component-will-receive-props": noComponentWillReceiveProps,
8381
"no-component-will-update": noComponentWillUpdate,

packages/plugins/eslint-plugin-react-x/src/rules/no-complex-conditional-rendering.md

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

0 commit comments

Comments
 (0)