jsx-eslint/eslint-plugin-react

Rule idea: no-mutation-props

Open

#1.113 geöffnet am 14. März 2017

Auf GitHub ansehen
 (6 Kommentare) (8 Reaktionen) (0 zugewiesene Personen)JavaScript (8.630 Stars) (2.797 Forks)batch import
help wantednew rulequestion

Beschreibung

A component is never meant to modify its own props (https://facebook.github.io/react/docs/components-and-props.html#props-are-read-only).

I suggest we had a no-mutation-props rule to check for mutations of a components own props. It looks like we could likely copy the logic used in no-direct-mutation-state.js.

Contributor Guide

Tech Stack
javascriptreactnodejs
Domain
toolingdeveloper experience
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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.
clear
Voraussetzungen
ESLint plugin developmentReact props immutabilityAST manipulation
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
Look at the existing rule no direct mutation state.js in the same repository (lib/rules/no direct mutation state.js) to understand the logic for detecting mutations. Adapt it to detect mutations of props instead of state. Consider edge cases like nested props and object spread. Review the React documentation link provided in the issue. Check the issue comments for any additional discussion on implementation details or scope.