Skip to content

Commit d3aa304

Browse files
authored
Fix spelling and grammar (DianaSuvorova#40)
* Fix spelling and grammar * Add feedback
1 parent 6a312ab commit d3aa304

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

docs/rules/mapStateToProps-no-store.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Enforces that mapStateToProps does not bind complete store to a component. (react-redux/mapStateToProps-no-store)
22

3-
Passing whole state to a component is a bd practice. Triggering unnecessary re-renders.
4-
Instead one should provide specific properties used by a component.
3+
Passing whole state to a component is a bad practice, triggering unnecessary re-renders. Additionally bad is passing around a mutable object that your component critically depends on preventing mutations to.
4+
Instead one should specify the properties actually used by a component.
55

66
## Rule details
77

@@ -45,7 +45,7 @@ connect((state) => ({isActive: state.isActive}), null)(App)
4545

4646
## Not supported use cases.
4747

48-
Please note that the following use case although common is not supported due to the nature of static code analysis.
48+
Please note that the following use case, although common, is not supported due to the nature of static code analysis.
4949

5050
The following would not warn:
5151

0 commit comments

Comments
 (0)