enb / enb-borschik

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

#14 Remove css-borschik-chunks technology #26

Closed tormozz48 closed 9 years ago

tormozz48 commented 9 years ago

Resolved #14

@blond

Please review it.

blond commented 9 years ago

Нельзя же просто так взять и удалить технологию :)

Предлагаю перенести её в enb-bembundle.

/сс @zxqfox

tormozz48 commented 9 years ago

Интересно, а этим вообще кто-то пользуется?

blond commented 9 years ago

Интересно, а этим вообще кто-то пользуется?

Нет никакой возможности проверить. Да и, перенести технологию дёшево.

qfox commented 9 years ago

@tormozz48 @blond кажется, что можно перепродумать технологию и унести в bembundle, если никто не отзовется.

blond commented 9 years ago

@zxqfox, подскажешь про как «перепродумать»? :)

qfox commented 9 years ago

Ну я в плане того, что borschik собирает, bembundle делит и пересобирает, конфликт интересов есть.

blond commented 9 years ago

Ну я в плане того, что borschik собирает, bembundle делит и пересобирает, конфликт интересов есть.

Вроде понятно, а как исходя из этого менять технологию нет :)

@zxqfox, как считаешь, просто скопипастить код в enb-bembundle не достаточно?

qfox commented 9 years ago

@blond ну если других вариантов нет, то да.

Я все еще хочу какую-то возможность препроцессинга до запуска основных технологий. Это бы сильно упростило процесс сборки. UPD: и кол-во технологий ;-)

blond commented 9 years ago

Я все еще хочу какую-то возможность препроцессинга до запуска основных технологий. Это бы сильно упростило процесс сборки. UPD: и кол-во технологий ;-)

@zxqfox Если тебе хоть как-то поможет, то мы начали покрывать ядро тестами — https://github.com/enb-make/enb/issues/256 :)

qfox commented 9 years ago

@blond Сильно поможет, попробую поучаствовать в июле ;-)

Тогда давайте пока унесем её в bembundle?

blond commented 9 years ago

Тогда давайте пока унесем её в bembundle?

ок! )

@tormozz48, Давай вначале добавим технологию в enb-bembundle, а тут объявим технологию deprecated. Удалим при выпуске мажорной версии.

qfox commented 9 years ago

Давай вначале добавим технологию в enb-bembundle, а тут объявим технологию deprecated. Удалим при выпуске мажорной версии.

:+1:

blond commented 9 years ago

:+1: