Skip to content
This repository has been archived by the owner on Jan 13, 2018. It is now read-only.

Feedback for "Настройка" #609

Open
wKich opened this issue Jan 28, 2016 · 2 comments
Open

Feedback for "Настройка" #609

wKich opened this issue Jan 28, 2016 · 2 comments

Comments

@wKich
Copy link

wKich commented Jan 28, 2016

getTechs() возвращает массив подключаемых технологий

exports.getTechs = function() {
    return {
        'bemjson.js': '',
        'js': 'js-i',
        'bemhtml.js': '../../bem-bl/blocks-common/i-bem/bem/techs/bemhtml.js',
        'priv.js': '../../.bem/techs/priv.js',
        'html': '../../bem-bl/blocks-common/i-bem/bem/techs/html'
    };
};

Неожиданно... { foo: 'bar' } - это массив, Карл. Массив!

Object.prototype.toString.call({}) //"[object Object]"
Object.prototype.toString.call([]) //"[object Array]"

Функция getConfig() возвращает объект со свойством bundleBuildLevels

exports.getConfig = function() {
    return extend({}, this.__base() || {}, {
        bundleBuildLevels: this.resolvePaths([
            '../../bem-bl/blocks-common',
            '../../bem-bl/blocks-desktop',
            '../../blocks'
        ])
    });
};

Стоп, у вас { } - это массив с ключами или объект со свойствами?

@tadatuta
Copy link
Contributor

@wKich конечно, в первом случае опечатка. Исправим, спасибо!

PS: Должен предупредить, что на данный момент рекомендуем сборщиком является ENB, а команда bem make из пакета bem-tools под капотом просто вызывает enb make с фоллбеком на старый bem make лишь из соображений обратной совместимости. Так что стоит сразу погружаться в сборку на ENB. И да, нам, конечно, нужно написать про это в самой документации, виноваты...

@wKich
Copy link
Author

wKich commented Jan 29, 2016

Большое спасибо за разъяснение.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants