Skip to content

Commit afcc62f

Browse files
committed
chore(eslint): disable some annoying rules
react/jsx-props-no-spreading: We spread props, a lot. It very often improperly adds attributes to DOM elements when it shouldn't, so I get why this rule exists. For now the strategy is to refactor spreading to NOT pass those attributes to DOM (see console messages for warnings), but not completely remove prop spreading. react/require-default-props: Default props are being removed from function components in the future, which throws warnings. So if we don't use default props anymore, then stop warning us when we don't have them.
1 parent 09bc667 commit afcc62f

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

.eslintrc.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,15 @@ rules:
2626
- ".jsx"
2727
react/jsx-one-expression-per-line:
2828
- 'off'
29+
react/jsx-props-no-spreading:
30+
- 'off'
2931
react/no-array-index-key:
3032
- 'off'
3133
react/no-danger:
3234
- 'off'
3335
react/no-typos:
3436
- 'off'
37+
react/require-default-props:
38+
- 'off'
3539
settings:
3640
import/external-module-folders: [ 'assets', 'config', 'ui', 'lib']

0 commit comments

Comments
 (0)