Linter przy commicie - Githubissues

michalbe / jsnews-client

:alien: Commandline client for JS News

http://github.com/michalbe/jsnews-client

MIT License

38 stars 5 forks source link

Linter przy commicie #42

Closed michalbe closed 10 years ago

michalbe commented 10 years ago

Ustawić hook na commit z gita i podpiąć pod linter.

leriel commented 10 years ago

Czy podmiana "if(" na "if (" nie bedzie przeszkadzać po stronie linterów których używacie? jshint to ignoruje ale closure linterowi sie to nie podoba

leriel commented 10 years ago

I analogiczne pytanie o podmianę "function (" na "function("

Maciek10 commented 10 years ago

Z tego co widze to jest zainstalowy precommit-hook ktory ma w sobie odpowiedni linter (ten sam co na travisie), wiec przy npm install powinien sie sam zainstalowac.

michalbe commented 10 years ago

Dokładnie tak, hook jest już zainstalowany i powinien niepozwolić commitować jeśli linter nie przejdzie. Żeby przelintować lokalnie w dowolnym momencie wystarczy uruchomić npm test. Zamykam issue.