jsx-eslint/eslint-plugin-react

Rule proposal: check constructor params for props and context

Open

#626 ouverte le 7 juin 2016

Voir sur GitHub
 (7 commentaires) (5 réactions) (0 assignés)JavaScript (8 630 stars) (2 797 forks)batch import
help wantednew rule

Description

This rule would check for:

class X extends Component {
  constructor(props, context) {
    super(props, context);
  }
}

React is lenient, so constructor() { super(); } works, so this is just a stylistic preference.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontenddeveloper experiencetooling
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaScriptReactESLint rule development
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Examine existing rules in the eslint plugin react repository (e.g., rules/no direct mutation state) for implementation patterns. Use the ESLint custom rule documentation to create a rule that checks the number and names of parameters in constructor of React components extending Component. The rule should be configurable for strict checking of props and context parameters. Consider edge cases like PureComponent or using constructor without super.