Open eidottermihi opened 6 years ago
Nachdem wir das Issue gelesen haben, stellen wir das jetzt doch zurück. Sonst müssten wir 1200 Errors fixen und das würden wir nicht so schnell hinbekommen und uns auch die Templates wieder zerschießen.
Idee (mit @Baumfrosch): Mit @tderflinger durchgehen
Das Thema wurde bereits mehrfach angesprochen: #241 , #254
Das Polymer Starter Kit verwendet zwei Tools fürs Linting:
Polymer Linter prüft gegen spezifische Regeln, was die Gestaltung/Implementierung von Polymer Web Components betrifft. Die Regeln können in der
polymer.json
konfiguriert werden:Das Polymer Linting kann einfach mit der polymer-cli via
polymer lint
aufgerufen werden.ESLint ist dann ein genereller "Linter" für JavaScript. Das Polymer Starter Kit inkludiert auch bereits eine angepasste Konfiguration für Polymer Projekte (
.eslintrc.json
).Ein "komplettes Linting" würde dann aus dem Aufruf beider Tools bestehen. Das Polymer Starter Kit legt dazu auch bereits ein "npm Skript" im
package.json
an:Ich habe mal beide Tools mit der "Standard-Konfiguration" über den aktuellen master gejagt: ESLint:
✖ 1297 problems (1297 errors, 0 warnings)
Polymer Lint:Found 23 errors 755 warnings. 1 can be automatically fixed with --fix.
Grundsätzlich zu klären ist aus meiner Sicht:
Zu 2.: Falls ja, müssten wir zumindest auf 0 Errors kommen :-)