Linter / Linting für Frontend

[eidottermihi]

Das Thema wurde bereits mehrfach angesprochen: #241 , #254

Das Polymer Starter Kit verwendet zwei Tools fürs Linting:

• ESLint
• Polymer Linter

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:

  "lint": {
    "rules": ["polymer-2"]
  }

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:

  "scripts": {
    "lint": "npm run lint:javascript && polymer lint",
    "lint:javascript": "eslint . --ext js,html --ignore-path .gitignore",
  }

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:

1. Welche Regel(-sätze) wollen wir anwenden (Diskussionsissue #241 erwähnt hier striktere Regeln)?
2. Wollen wir das Linting in die Build Pipeline aufnehmen?

Zu 2.: Falls ja, müssten wir zumindest auf 0 Errors kommen :-)

[rowe42]

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.

[rowe42]

Idee (mit @Baumfrosch): Mit @tderflinger durchgehen