Commit 9b5001e
committed
[eslint] Add an option to require dependencies on effect hooks (#33344)
Summary:
To prepare for automatic effect dependencies, some codebases may want to
codemod
existing useEffect calls with no deps to include an explicit undefined
second argument
in order to preserve the "run on every render" behavior. In sufficiently
large codebases,
this may require a temporary enforcement period where all effects
provide an explicit
dependencies argument.
Outside of migration, relying on a component to render can lead to real
bugs,
especially when working with memoization.
DiffTrain build for [99efc62](99efc62)1 parent 556b910 commit 9b5001e
File tree
27 files changed
+116
-89
lines changed- compiled-rn
- facebook-fbsource/xplat/js
- RKJSModules/vendor/react
- react-dom/cjs
- react-test-renderer/cjs
- react/cjs
- react-native-github/Libraries/Renderer
- implementations
- tools/eslint-plugin-react-hooks
- cjs
27 files changed
+116
-89
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
404 | 404 | | |
405 | 405 | | |
406 | 406 | | |
407 | | - | |
| 407 | + | |
408 | 408 | | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
27026 | 27026 | | |
27027 | 27027 | | |
27028 | 27028 | | |
27029 | | - | |
| 27029 | + | |
27030 | 27030 | | |
27031 | 27031 | | |
27032 | 27032 | | |
27033 | | - | |
| 27033 | + | |
27034 | 27034 | | |
27035 | 27035 | | |
27036 | 27036 | | |
| |||
27067 | 27067 | | |
27068 | 27068 | | |
27069 | 27069 | | |
27070 | | - | |
| 27070 | + | |
27071 | 27071 | | |
27072 | 27072 | | |
27073 | | - | |
| 27073 | + | |
27074 | 27074 | | |
27075 | 27075 | | |
27076 | 27076 | | |
| |||
27208 | 27208 | | |
27209 | 27209 | | |
27210 | 27210 | | |
27211 | | - | |
| 27211 | + | |
27212 | 27212 | | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
17094 | 17094 | | |
17095 | 17095 | | |
17096 | 17096 | | |
17097 | | - | |
| 17097 | + | |
17098 | 17098 | | |
17099 | 17099 | | |
17100 | 17100 | | |
17101 | 17101 | | |
17102 | 17102 | | |
17103 | 17103 | | |
17104 | | - | |
| 17104 | + | |
17105 | 17105 | | |
17106 | 17106 | | |
17107 | 17107 | | |
| |||
17123 | 17123 | | |
17124 | 17124 | | |
17125 | 17125 | | |
17126 | | - | |
| 17126 | + | |
17127 | 17127 | | |
17128 | 17128 | | |
17129 | | - | |
| 17129 | + | |
17130 | 17130 | | |
17131 | 17131 | | |
17132 | 17132 | | |
| |||
17224 | 17224 | | |
17225 | 17225 | | |
17226 | 17226 | | |
17227 | | - | |
| 17227 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
17804 | 17804 | | |
17805 | 17805 | | |
17806 | 17806 | | |
17807 | | - | |
| 17807 | + | |
17808 | 17808 | | |
17809 | 17809 | | |
17810 | 17810 | | |
17811 | 17811 | | |
17812 | 17812 | | |
17813 | 17813 | | |
17814 | | - | |
| 17814 | + | |
17815 | 17815 | | |
17816 | 17816 | | |
17817 | 17817 | | |
| |||
17833 | 17833 | | |
17834 | 17834 | | |
17835 | 17835 | | |
17836 | | - | |
| 17836 | + | |
17837 | 17837 | | |
17838 | 17838 | | |
17839 | | - | |
| 17839 | + | |
17840 | 17840 | | |
17841 | 17841 | | |
17842 | 17842 | | |
| |||
17949 | 17949 | | |
17950 | 17950 | | |
17951 | 17951 | | |
17952 | | - | |
| 17952 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
27082 | 27082 | | |
27083 | 27083 | | |
27084 | 27084 | | |
27085 | | - | |
| 27085 | + | |
27086 | 27086 | | |
27087 | 27087 | | |
27088 | 27088 | | |
27089 | | - | |
| 27089 | + | |
27090 | 27090 | | |
27091 | 27091 | | |
27092 | 27092 | | |
| |||
27123 | 27123 | | |
27124 | 27124 | | |
27125 | 27125 | | |
27126 | | - | |
| 27126 | + | |
27127 | 27127 | | |
27128 | 27128 | | |
27129 | | - | |
| 27129 | + | |
27130 | 27130 | | |
27131 | 27131 | | |
27132 | 27132 | | |
| |||
27580 | 27580 | | |
27581 | 27581 | | |
27582 | 27582 | | |
27583 | | - | |
| 27583 | + | |
27584 | 27584 | | |
27585 | 27585 | | |
27586 | 27586 | | |
| |||
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
17105 | 17105 | | |
17106 | 17106 | | |
17107 | 17107 | | |
17108 | | - | |
| 17108 | + | |
17109 | 17109 | | |
17110 | 17110 | | |
17111 | 17111 | | |
17112 | 17112 | | |
17113 | 17113 | | |
17114 | 17114 | | |
17115 | | - | |
| 17115 | + | |
17116 | 17116 | | |
17117 | 17117 | | |
17118 | 17118 | | |
| |||
17134 | 17134 | | |
17135 | 17135 | | |
17136 | 17136 | | |
17137 | | - | |
| 17137 | + | |
17138 | 17138 | | |
17139 | 17139 | | |
17140 | | - | |
| 17140 | + | |
17141 | 17141 | | |
17142 | 17142 | | |
17143 | 17143 | | |
| |||
17388 | 17388 | | |
17389 | 17389 | | |
17390 | 17390 | | |
17391 | | - | |
| 17391 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
17819 | 17819 | | |
17820 | 17820 | | |
17821 | 17821 | | |
17822 | | - | |
| 17822 | + | |
17823 | 17823 | | |
17824 | 17824 | | |
17825 | 17825 | | |
17826 | 17826 | | |
17827 | 17827 | | |
17828 | 17828 | | |
17829 | | - | |
| 17829 | + | |
17830 | 17830 | | |
17831 | 17831 | | |
17832 | 17832 | | |
| |||
17848 | 17848 | | |
17849 | 17849 | | |
17850 | 17850 | | |
17851 | | - | |
| 17851 | + | |
17852 | 17852 | | |
17853 | 17853 | | |
17854 | | - | |
| 17854 | + | |
17855 | 17855 | | |
17856 | 17856 | | |
17857 | 17857 | | |
| |||
18117 | 18117 | | |
18118 | 18118 | | |
18119 | 18119 | | |
18120 | | - | |
| 18120 | + | |
18121 | 18121 | | |
18122 | 18122 | | |
18123 | 18123 | | |
| |||
0 commit comments