[Proposal] Enabling `no-param-reassign` eslint rule
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 122k
- Forks
- 37.4k
- Merge moyen
- 4 j 3 h
- PR mergées (30 j)
- 272
Description
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).
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par examiner la règle ESLint proposée no-param-reassign et exécuter les vérifications ESLint existantes du dépôt. L’exécution actuelle du linter signale 1 597 erreurs ; le travail est terminé lorsque la règle est activée et que ces violations de réaffectation de paramètres sont corrigées sans introduire de nouvelles erreurs de lint.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- tooling
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100