[Proposal] Enabling `no-param-reassign` eslint rule
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 122k
- Forks
- 37.3k
- Ø Merge
- 4 T. 2 Std.
- Gemergte PRs (30 T.)
- 283
Beschreibung
I've personally often feel frustrated due to being extremely hard to track an implementation after a reassignment of parameters, due to the ambiguity of types in a function that has a JSDoc declaration.
I recommend enabling https://eslint.org/docs/latest/rules/no-param-reassign. There is currently ✖ 1597 problems (1597 errors, 0 warnings).
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die vorgeschlagene ESLint-Regel no-param-reassign zu überprüfen und die vorhandenen ESLint-Prüfungen des Repositorys auszuführen. Der aktuelle Lint-Lauf meldet 1.597 Fehler; als erledigt gilt die Aufgabe, wenn die Regel aktiviert ist und die Verstöße gegen die Neuzuweisung von Parametern behoben sind, ohne neue Lint-Fehler einzuführen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100