search

enb / enb-borschik

ENB tech to run borschik
Other
13 stars 15 forks source link

Rewrite borschik technology with help of build-flow module #23

Closed tormozz48 closed 9 years ago

tormozz48 commented 9 years ago

Resolved #20

@blond @tavriaforever @sipayRT Please review it

tavriaforever commented 9 years ago

Вопрос может не совсем по тему этого PR, но в тестах заметил, что есть тест на использования cleancss, и дополнительных require этой штуки нет и тесты как-то проходят, получается, что доп зависимость от borschik-tech-cleancss не нужна или это не честный тест? Ведь даже без cleancss результат будет такой же практически.

blond commented 9 years ago

Вопрос может не совсем по тему этого PR, но в тестах заметил, что есть тест на использования cleancss, и дополнительных require этой штуки нет и тесты как-то проходят, получается, что доп зависимость от borschik-tech-cleancss не нужна или это не честный тест? Ведь даже без cleancss результат будет такой же практически.

У борщика такое API, что можно указать имя технологии, и он будет искать пакет borschik-tech-<имя технологии>.

tavriaforever commented 9 years ago

Ну ок, тогда что мы тут проверяем?

blond commented 9 years ago

Ну ок, тогда что мы тут проверяем?

Что борщик впринципе работает, опции передаются и т.д. Проверять все кейсы, скажем, как в enb-stylus тут смысла нет.

sipayRT commented 9 years ago

:+1:

blond commented 9 years ago

@tormozz48, соскваш коммиты, пожалуйста.

tormozz48 commented 9 years ago

объединил коммиты в 1

tavriaforever commented 9 years ago

Мне ок