jsx-eslint/eslint-plugin-react

react/prefer-stateless-function works different than described in docs

Open

#1 901 ouverte le 26 juil. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (8 630 stars) (2 797 forks)batch import
bughelp wanted

Description

For the react/prefer-stateless-function rule the docs for ignorePureComponents option states:

When true the rule will ignore Components extending from React.PureComponent that use this.props or this.context.

Actually it also ignore every PureComponent that use not only props or context, but ignores every component that use this. If this is not intended I prepared a fix (GH-1902), otherwise it makes sense to update the docs.

Guide contributeur

Stack technique
javascriptreact
Domaine
developer experiencetooling
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
ESLint rule developmentReact component patterns
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Examine the rule implementation in lib/rules/prefer stateless function.js and its test suite to confirm the behavior discrepancy. Check the linked PR (GH 1902) for a proposed code fix and ensure it aligns with the intended rule logic. Evaluate whether updating the documentation to match current behavior is simpler than fixing the code. Since the issue is old and has no recent activity, consider reaching out to maintainers for guidance on the preferred resolution path.