bem-site / bem-forum-content-ru

Content BEM forum for Russian speak users
MIT License
56 stars 6 forks source link

Не срабатывает настройки uglify у борщика #560

Open pavelpower opened 9 years ago

pavelpower commented 9 years ago

Может кто подскажет по чему?

[borschikJs, {
                    sourceTarget: '?.js',
                    destTarget: '_?.js',
                    techOptions: {
                        uglify: {
                            quote_style: 1
                        }
                    },
                    minify: true,
                    freeze: true,
                    noCache: true
                }]
pavelpower commented 9 years ago

Похоже на то, что у борщика очень старая либа uglify-js

VS

tadatuta commented 9 years ago

Завел https://github.com/bem/borschik/issues/108 Пришлешь pull request? ;)

Guria commented 9 years ago

https://github.com/bem/borschik/issues/106

pavelpower commented 9 years ago

https://github.com/bem/borschik/issues/106

PR готов: https://github.com/bem/borschik/pull/107

pavelpower commented 9 years ago

Проверил работает, если настроить так:

                 [borschikJs, {
                    sourceTarget: '?.js',
                    destTarget: '_?.js',
                    tech: 'js', // <------------- указать обязательно
                    techOptions: {
                        uglify: {
                            output: { // <------------- от такая вот настройка
                                quote_style: 3
                            }
                        }
                    },
                    minify: true,
                    freeze: true,
                    noCache: true
                }]
pavelpower commented 9 years ago

@tadatuta, извини, с таском чуть опередил :)

blond commented 9 years ago

@pavelpower, какая версия enb-borschik приезжала? В v1.5.1 такая ошибка правилась.

pavelpower commented 9 years ago

@blond это не ошибка, просто нужно обновить либу uglify2 до версии 2.4.24. В данном ПР для борщика прьблема решается.

В enb-borschik сделано все ок, т.к. используется крайняя версия борщика "borschik": "^1.3.2".

pavelpower commented 9 years ago

@blond, пока ПР не принят используем грязный хак

npm install https://github.com/pavelpower/borschik/archive/v1.4.2.tar.gz --save