jsx-eslint/eslint-plugin-react

no-direct-mutation-state on references

Open

#747 aperta il 7 ago 2016

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (2797 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (8630 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Currently no-direct-mutation-state doesn't seem to support hunting down (even simple cases) where a reference to this.state is being used. Luckily I caught my co-workers bug while doing code review but it would be pretty sweet if the rule could detect this case too.

const { product } = this.state;
product.name = name;

I've never written an eslint rule before so I have no clue if this is even possible, if it is, please let me know and I might give it a stab when I have some more free time.

Guida contributor

Direzione di ricerca
Ispeziona il codice sorgente della regola in eslint plugin react per capire come rileva attualmente le mutazioni dirette dello stato. Poi estendila per rilevare anche mutazioni tramite riferimenti come `const { product } = this.state; product.name = name;`.
Tech stack
javascriptreact
Dominio
frontendtooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptESLint rule development
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65