Less.js: [Wartungsaufgabe] - Überprüfen Sie die ESLint-Einstellungen und beheben Sie Fusselprobleme
Vor einiger Zeit habe ich das Projekt von JSHint auf ESLint umgestellt (letzteres ist besser gewartet und bietet mehr Funktionen). Ich habe jedoch viele der Einstellungen als Warnungen festgelegt, da ich keine Zeit hatte, potenzielle Probleme zu beheben. Ich hatte auch nicht wirklich Zeit, Linting für Einstellungen zu optimieren, die den Code wartungsfreundlicher machen.
Die Aufgabe besteht also im Wesentlichen darin, sich die ESLint-Warnungen anzusehen und zu sehen, was sicher behoben werden kann. (Viele der == Gleichheitswarnungen können beispielsweise nicht auf === "korrigiert" werden, ohne für jeden Wert zu testen, der diese Prüfung durchläuft.) Es wäre hilfreich, auch Warnungen für . festzulegen Dinge, die in Zukunft / fortlaufend angegangen werden müssen, wie z. B. die Durchsetzung der JSDoc-Dokumentation.
matthew-dean
Alle 6 Kommentare
Hallo zusammen, das würde ich gerne greifen!
umuur
am 18. Jan. 2021
@umuur Mach es!
matthew-dean
am 19. Jan. 2021
@matthew-dean Wollen wir die aktuelle eslint-Konfiguration beibehalten?
Ich bin mir nicht sicher, ob TypeScript als Parser und Plugin erforderlich ist.
umuur
am 19. Jan. 2021
@umuur
Möchten wir die aktuelle eslint-Konfiguration beibehalten?
Es hängt davon ab, ob. Wenn Sie Änderungen vornehmen möchten, die die Codeformatierung nicht ändern, ist das in Ordnung. Aber was Warnungen angeht, sind das IMO alles gültige Warnungen und wir möchten sie entweder ansprechen oder einen Code-Kommentar zu deaktiviertem Eslint und ( am wichtigsten ) hinterlassen, warum es dort nicht gültig ist.
Möchten wir die aktuelle eslint-Konfiguration beibehalten?
Obwohl die Codebasis TypeScript noch nicht verwendet, leistet sie meiner Erfahrung nach beim Transpilieren eine bessere Arbeit als Babel. In Bezug auf ESLint .... 🤔 Ja, Sie haben Recht, es wird technisch nicht benötigt, bis es Code in TypeScript gibt. Ich habe das wahrscheinlich in Erwartung der Konvertierung der Codebasis in TS getan, aber jetzt weiß ich nicht, ob / wann das passieren wird.
matthew-dean
am 20. Jan. 2021
@umuur
Übrigens, überall, wo Sie JSDoc-Kommentare mit den richtigen Typen zu Parametern hinzufügen können, fügen Sie sie bitte hinzu!
matthew-dean
am 20. Jan. 2021
@matthew-dean danke für die ausführlichen Kommentare! Werde Sie über die Updates informieren.
umuur
am 20. Jan. 2021
Verwandte Themen
falkenhawk
·
5Kommentare
pknepper
·
3Kommentare
BrianMulhall
·
4Kommentare
vecerek
·
5Kommentare
awebdev
·
4Kommentare