jsx-eslint/eslint-plugin-react

react/no-direct-mutation-state doesn't catch all cases

Open

#1.418 geöffnet am 8. Sept. 2017

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.797 Forks)batch import
help wanted

Repository-Metriken

Stars
 (8.630 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

  const { model } = this.state;
  model.rating = rate;

In this case eslint won't show any errors.

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung der Regel no direct mutation state, um zu verstehen, warum sie Mutationen bei der Destrukturierung des Zustands nicht erfasst. Sehen Sie sich Testfälle für vorhandene Muster an und fügen Sie einen Testfall für destrukturierte Zuweisungen hinzu. Ändern Sie dann die AST Traversierung der Regel, um Mutationen an Objekteigenschaften zu erkennen, die durch Destrukturierung gewonnen wurden.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Fehler
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 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ESLintReact
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60