jsbench / jsbench.github.io

JavaScript benchmark playground
http://jsbench.github.io/
MIT License
266 stars 13 forks source link

Linting v.2 #5

Closed nicksp closed 8 years ago

nicksp commented 8 years ago

Более-менее финальный линт для проекта.

Я перебрал каждую опцию, поигрался с каждым параметром. Постарался сделать конфиг согласно рекомендациям airbnb.

Инструмент действительно крутой. Позволяет столько всего проверять -- очень гибкий и настраиваемый. Довольно подробная документация со множеством примеров и чистый консольный вывод. Видно какой параметр зафейлился и где.

Также можно задавать уровень ошибки и тонкую подстройку опции: 0 (off), 1 (warning) или 2 (critical error).

Кроме этого почистил package.json: удалил неиспользуемые модули.

Поправил сразу сорцы согласно логу. Надеюсь, не зачмыришь :)

P.S. Если запустишь gulp lint, то увидишь 2 ворнинга ( vars-on-top и no-undefined). Оставил преднамеренно, чтобы ты поправил как надо: либо отключить проверку совсем, либо изменить в коде.

P.P.S. Опции, специфичные для ES6, добавлю в следующем Пул Реквесте, где буду заниматься переводом проекта на ES6 и сборкой бандла (предположительно четверг-пятница).

RubaXa commented 8 years ago

Отлично, поправь только объявления, слишком высоко унес ;]

nicksp commented 8 years ago

@RubaXa Хорошо. Сегодня поправлю.

RubaXa commented 8 years ago

:+1:

nicksp commented 8 years ago

Nice :)