Skip to content

Commit ed56e53

Browse files
authored
Revert "New rule no-access-state-in-setstate (#190)"
This reverts commit 987a95e.
1 parent 987a95e commit ed56e53

File tree

4 files changed

+0
-206
lines changed

4 files changed

+0
-206
lines changed

README.md

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ The built-in configuration preset you get with `"extends": "tslint-react"` is se
5252
size={size}
5353
/>
5454
```
55-
- Rule options: _none_
5655
- `jsx-ban-elements` (since v3.4.0)
5756
- Allows blacklisting of JSX elements with an optional explanatory message in the reported failure.
5857
- `jsx-ban-props` (since v2.3.0)
@@ -122,22 +121,6 @@ The built-in configuration preset you get with `"extends": "tslint-react"` is se
122121
</button>
123122
);
124123
```
125-
- Rule options: _none_
126-
- `no-access-state-in-setstate`
127-
- Forbids accessing component state with `this.state` within `this.setState`
128-
calls, since React might batch multiple `this.setState` calls, thus resulting
129-
in accessing old state. Enforces use of callback argument instead.
130-
```ts
131-
// bad
132-
this.setState({
133-
counter: this.state.counter + 1
134-
});
135-
// good
136-
this.setState(
137-
prevState => ({ counter: prevState.counter + 1 })
138-
);
139-
```
140-
- Rule options: _none_
141124

142125
### Development
143126

src/rules/noAccessStateInSetstateRule.ts

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

test/rules/no-access-state-in-setstate/test.tsx.lint

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

test/rules/no-access-state-in-setstate/tslint.json

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

0 commit comments

Comments
 (0)