airbnb / airbnb/javascript

Ignore property modifications on params named "state"

Aperta
#2,485 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
148k
Fork
26.6k
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

The [`no-param-reassign` rule](https://github.com/airbnb/javascript/blob/4ee732639396362c6981b1aa4730934c3baa7919/packages/eslint-config-airbnb-base/rules/best-practices.js#L228-L246) allows property modifications for params with certain names, like `acc` and `response`. I would like to suggest that `state` is also allowed, for users of Redux Toolkit. The Redux Toolkit documentation [recommends this](https://redux-toolkit.js.org/usage/immer-reducers#linting-state-mutations).

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia in packages/eslint-config-airbnb-base/rules/best-practices.js, nella configurazione no-param-reassign collegata, e confronta i nomi dei parametri attualmente consentiti. Aggiorna la configurazione in modo da consentire le modifiche alle proprietà dei parametri denominati "state", quindi verifica il comportamento della regola per gli aggiornamenti dello stato in stile Redux Toolkit.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
eslint, javascript
Ambito
tooling
Tipo di issue
Funzionalità
Difficoltà
1/5
Tempo stimato
Meno di un'ora
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.