search

enb / enb-borschik

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

Issue 18 Implement tests for borschik technology #21

Closed tormozz48 closed 9 years ago

tormozz48 commented 9 years ago

Resolved #18

@blond @sipayRT @tavriaforever

Please review it

blond commented 9 years ago

Можно добавть тест с прокидыванием опций для js, например, так:

techOptions: {
    uglify: {
        output: {
            max_line_len: 250
        }
    }
}
blond commented 9 years ago

Написал несколько замечаний, в целом :ok:

sipayRT commented 9 years ago

:+1:

blond commented 9 years ago

:+1:

tormozz48 commented 9 years ago

Можно добавть тест с прокидыванием опций для js, например, так:

А у нас нет пакета borschik-tech-uglify :(

blond commented 9 years ago

А у нас нет пакета borschik-tech-uglify :(

Он и не нужен, опцию tech указывать не надо. Это тест для родной технологии.

blond commented 9 years ago

А сам borschik умеет работать под Windows?

tormozz48 commented 9 years ago

А сам borschik умеет работать под Windows?

А это хороший вопрос коллега :)

blond commented 9 years ago

Если нет, то давай просто не будет запускать тесты под Windows для этого пакета. Судя по логу ошибок, там как раз проблема не в том, что слеши не равильные или переносы строк, а что код просто не работает.

tormozz48 commented 9 years ago

Ок, давай пока отключим

tormozz48 commented 9 years ago

@blond Давай наверное вливать. P.S. Я добавил еще один тест с uglify опциями.

blond commented 9 years ago

Конфиг для appveyor не удалил же.

tormozz48 commented 9 years ago

Я думал, тестирование просто в appveyor отключается какой-то кнопкой, как в трэвисе