jsx-eslint/eslint-plugin-react

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

Open

#1.901 geöffnet am 26. Juli 2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (8.630 Stars) (2.797 Forks)batch import
bughelp wanted

Beschreibung

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.

Contributor Guide

Tech Stack
javascriptreact
Domain
developer experiencetooling
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
ESLint rule developmentReact component patterns
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
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.