Index module should provide functions instead of techs

[blond]

Сейчас конфиг пестрит обилием квадратных скобок.

var techs = require('enb-bem-techs');

nodeConfig.addTechs([
    [techs.levels, { levels: ['blocks'] }],
    [require('enb/techs/file-provider'), { target: '?.bemjson.js' }],
    [techs.bemjsonToBemdecl],
    [techs.depsOld],
    [techs.files],
    /* ... */
]);

Можно написать небольшую функцию-враппер, которая будет превращать технологию в функцию, возвращающую массив из модуля технологии и указанных опций.

var bem = require('enb-bem-techs'),
    useTech = require('enb-bem-techs/lib/use-tech'),
    provide = useTech(require('enb/techs/file-provider'));

nodeConfig.addTechs([
    bem.levels({ levels: ['blocks'] }),
    provide({ target: '?.bemjson.js' }),
    bem.bemjsonToBemdecl(),
    bem.depsOld(),
    bem.files(),
    /* ... */
]);

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

var bem = require('enb-bem-techs'),
    useTech = require('enb-bem-techs/lib/use-tech'),
    provide = useTech(require('enb/techs/file-provider'));

nodeConfig.addTechs([
    bem.levels(['blocks']),
    provide('?.bemjson.js'),
    bem.bemjsonToBemdecl(),
    bem.depsOld(),
    bem.files(),
    /* ... */
]);

[blond]

/сс @tadatuta @arikon

[tadatuta]

так, конечно, выглядит красивее, но не боишься сделать изучение инструмента слишком сложным из-за обилия абстракций?

[arikon]

Можно это делать опциональным

31.10.2014, 23:00, "Vladimir Grinenko" [email protected]:

так, конечно, выглядит красивее, но не боишься сделать изучение инструмента слишком сложным из-за обилия абстракций?

—
Reply to this email directly or view it on GitHub.

Отправлено из мобильной Яндекс.Почты: http://m.ya.ru/ymail

[blond]

@arikon, расскажи, пожалуйста, подробнее как реализовать это опционально?

[arikon]

@andrewblond

• оставить возможность подключать модули технологий через require()
• оставить возможность передать хешик со всеми параметрами в каждую технологию

[blond]

оставить возможность подключать модули технологий через require()

Это всегда можно делать через require('enb-bem-techs/techs/tech-name');

Но получить объект с технологиями через var techs = require('enb-bem-techs') будет нельзя.

оставить возможность передать хешик со всеми параметрами в каждую технологию

Само собой.

[blond]

Предлагаю поддержать оба варианта подключения технологий:

1. require('enb-bem-techs/techs') — получаем объект с модулями технологий.
2. require('enb-bem-techs') — получаем объект с функциями-технологиями.

[tadatuta]

мой поинт был в том, что вводится новая абстракция всего лишь ради избавления от скобочек.
решение с возможность писать и со скобочками и без, кмк, эту проблему не решает, а только усиливает — в инструменте появляется все больше сущностей, которые нужно изучить, чтобы начать пользоваться. вы уверены, что оно того стоит?

[blond]

@tadatuta Кажется, что для пользователей, которые просто используют технологии абсолютно всё равно как их вызывать, и как технологии работают внутри, лижбы всё было описано в примерах и везде работало одинаково.

Для тех же кто сам пишет технологии не составит проблем понять как это работает.

[blond]

После обсуждения с @mdevils решили отложить до лучших времён =)

[blond]

Возможно лучшие времена настали? =)

Мы можем добавить подобное поведение «сбоку», положив подобные сахарные функции в файл отдельный файл.

var techs = require('enb-bem-techs'); // techs
var sugar = require('enb-bem-techs/sugar'); // sugar functions

[qfox]

@andrewblond Я думал, что мы это попробуем сделать в bem-components, если не пойдет — нафик. Но если хочешь более основательно — я не против ;-) Но есть риск, что опять скатимся в дискуссии о попугаях, которых сложно посчитать.

вы уверены, что оно того стоит?

@tadatuta Ради читаемости конфига — 100% да.

[tormozz48]

@blond Насколько эта задача актуальна?
P.S. Я понимаю как это сделать, но много ньюансов насчет засахаривания опций и.т.д.