rules-partii.md

ESLint Rules

This is for the second stage of linting rules.

Higher Priority

React-Specific

• react/no-access-state-in-setstate: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-access-state-in-setstate.md
• react/sort-comp: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/sort-comp.md TODO

Lower Priority

React-Specific

• react/boolean-prop-naming: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/boolean-prop-naming.md
• react/no-array-index-key: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-array-index-key.md TODO
• react/require-default-props: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/require-default-props.md TODO (?)
  • I know there was some push-back on this, but I still think it's worth it, considering that this reduces manual type-checking, provides a clearer API, and forces development to consider reusability.

Rules to Consider

React-Specific

• react/prefer-stateless-function: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/prefer-stateless-function.md
  • We should probably do this if we believe that functional components will have performance benefits.
• react/void-dom-elements-no-children: https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/void-dom-elements-no-children.md