jsnews-client

jsnews-client copied to clipboard

Jeśli nie macie nic przeciwko, chciałbym do jshint dodać closure linter (gjslint), podobnie jak w #42 w pre-commit

Poniższy screenshot pokazuje czego gjslint jeszcze się czepia + spacje między parametrami + koniec pustej linii na końcu

Są to rzeczy których nie znalazłem we flagach jshint (najbliżej była nieużywana już flaga white).

Główne pytanie to czy według Was kontrola lintera nad takimi rzeczami jest OK, czy to za dużo.

Jeśli uznacie że to za dużo, alternatywą jest dorzucenie do repo plik .gjslintrc do użytku dla osób korzystających z gjslint

Wydaje mi się że uzywanie 2 linterów to głupota, jeśli jshint nie ma odpowiednich flag (wydaje mi się ze krzyczał na spacje na końcu linii kiedyś) to może jest jakiś prosty sposób na zaimplementowanie gjslint w precommit hookach?

Tutaj jest wlasnie osobny problem, gjslint nie jest npmowskim narzędziem, a jedyna paczka z osadzonym narzędziem jaką znalazłem, nie byla update'owana przez rok i ma nieaktualną wersję

Śledząc historię issues jshint, wyczytałem że odeszli od kontrolowania stylu kodu wyrzucając w procesie niektóre oryginalne flagi i skupiając się bardziej na błędogennych rzeczach

Z kolei jslint który krzyczy dużo więcej i jest w miarę aktualny na npmie, ma parę mocno wkurzających rzeczy do których nie ma jeszcze flag

W związku z tym, temat na razie zostawiam, a w wolnej chwili zrobię zmiany jak na obrazku powyżej