Less.js: [Tâche de maintenance] - Examinez les paramètres ESLint et résolvez les problèmes de peluchage

Créé le 21 juin 2018 · 6Commentaires · Source: less/less.js

Il y a quelque temps, j'ai basculé le projet de JSHint vers ESLint (ce dernier étant mieux entretenu et offrant plus de fonctionnalités). Cependant, j'ai défini de nombreux paramètres comme des avertissements car je n'avais pas le temps de résoudre les problèmes potentiels. Je n'ai pas non plus vraiment eu le temps de peaufiner le linting pour des paramètres qui rendent le code plus facile à gérer.

Ainsi, la tâche consisterait essentiellement à examiner les avertissements ESLint et à voir ce qui peut être résolu en toute sécurité. (Par exemple, de nombreux avertissements d'égalité == peuvent ne pas être "réparables" à === sans tester pour chaque valeur qui passe par cette vérification.) Il serait utile de définir également des avertissements pour les choses à régler à l'avenir/en cours, comme l'application de la documentation JSDoc.

good first issue medium priority up-for-grabs

Source

matthew-dean

Tous les 6 commentaires

Salut, j'aimerais prendre ça !

umuur le 18 janv. 2021

@umuur Vas-y !

matthew-dean le 19 janv. 2021

👍1

@matthew-dean Voulons-nous conserver la configuration actuelle d'eslint ?
Je ne suis pas sûr qu'il soit nécessaire d'avoir TypeScript comme analyseur et plugin.

umuur le 19 janv. 2021

@umuur

Voulons-nous conserver la configuration actuelle d'eslint ?

Ça dépend. Si vous souhaitez apporter des modifications qui ne modifient pas le formatage du code, ce n'est pas un problème. Mais en ce qui concerne les avertissements, IMO, ce sont tous des avertissements valides et nous voulons soit les traiter, soit laisser un commentaire de code sur eslint désactivé et ( plus important encore ) pourquoi il n'est pas valide là-bas.

Voulons-nous conserver la configuration actuelle d'eslint ?

Même si la base de code n'utilise pas encore TypeScript, d'après mon expérience, elle fait un meilleur travail de transpilation que Babel. En termes d'ESLint.... oui, vous avez raison, ce n'est techniquement pas nécessaire tant qu'il n'y a pas de code dans TypeScript. Je l'ai probablement fait en prévision de la conversion de la base de code en TS, mais maintenant je ne sais pas si / quand cela se produira.

matthew-dean le 20 janv. 2021

👍1

@umuur

Au fait, partout où vous pouvez ajouter des commentaires JSDoc avec les types appropriés sur les paramètres, veuillez les ajouter !

matthew-dean le 20 janv. 2021

@matthew-dean merci pour les commentaires détaillés ! Vous informera des mises à jour.

umuur le 20 janv. 2021

👍1

Cette page vous a été utile?

0 / 5 - 0 notes