deps: possibility to declare dependence without explicit including item

[veged]

close #459

[scf2k]

@veged в v2 это не актуально?

[veged]

@scf2k актуально, щас добавим

[arikon]

@veged @dfilatov Объясните, пожалуйста, суть этих изменений. Из исходного таска я её не понял.

[arikon]

#401 уже закрыли, нужно раскомметировать?

[dfilatov]

@arikon вроде пока не закрыли

[arikon]

Ага, закроем завтра

[arikon]

@dfilatov

Это позволяет указывать зависимости между сущностями, но без подключения самих сущностей. Например, блок B не зависит от кода блока A напрямую, но если вдруг какой-то другой блок будет явно зависеть от A, то его код (код блока A) должен подключаться раньше подключения кода B. Для этого в B пишется такая зависимость (с ключом include: false), которая влияет только на порядок, без явного подключения самой сущности.

[dfilatov]

@arikon поправили с учетом твоих замечаний

[arikon]

@dfilatov Сделайте то же самое в v1 версии, пожалуйста

[veged]

перенёс каменты в v1 версию технологий и в пуллреквест про v1 версию bem-tools #464

[arikon]

/cc @SevInf @diunko

[arikon]

LGTM

[arikon]

@veged @dfilatov Здесь не хватает коммита про актуализацию эталонных результатов сборки

[veged]

@arikon тесты жеж проходят... я считал, что в этой ветке просто нет того теста

[arikon]

Окей

[arikon]

@veged @dfilatov Покажите пример блока, в котором используется include: false? Нам для понимания, в каких случаях рекомендуется это использовать.

[veged]

представьте, что есть блок input и у него есть 1) разные темы input_theme_{dark,light} и 2) опциональный элемент input__label

в стилях для темы написано переопределение базовых стилей для лейбла, поэтому input_theme_{dark,light} должен идти после input__label -- т.е. нужен mustDeps: input_theme_{dark,light} mustDeps input__label, но если так написать сейчас, то всегда с темой будет подцепляться input__label (т.к. у нас понятия "это должно идти перед этим" и "это должно попадать в финальную сборку вместе с этим" не разделены)

чтобы можно было указать зависимость по порядку следования в сборке, но при этом не требовать обязательного подключения опционального элемента мы сделали include: false -- input_theme_{dark,light} mustDeps input__label include: false -- в этом случае мы запомним факт про зависимость, но не будем подключать input__label до тех пор, пока кто-то его не попросит без include: false

[arikon]

@veged Спасибо за подробное описание!

@tadatuta Вова, это описание нужно унести в статью про deps.js на bem.info, когда вольём.

[arikon]

@veged К сожалению, Travis лежит. Надеюсь к завтра оклемается и мы сможем узнать финальный результат сборки.

[tadatuta]

Чарльз писал, что mustDeps и shouldDeps непонятны для англоговорящих (да и для русскоговорящих, кажется, тоже).

а mustDeps с include: false — это дополнительный взрыв мозга.

вопрос/предложение: может быть, пока не вышли bem-tools 1.0.0 и bem-core официально не зарелизился, добавим понятные алиасы, как обсуждали в переписке, и пореплейсим везде?

[veged]

@tadatuta мне кажется, что это всё-таки не связанные вещи -- этот пуллреквест опирается на текущую ситуацию, про переименование можно отдельно обсуждать

include: false не особо что-то усложняет и не подразумевается к масштабному использованию

[tadatuta]

@veged согласен, вопрос был не в контексте конкретного пулл-реквеста.

но то, что mustDeps (читается примерно как "должен зависит") при использовании вместе с include: false на самом деле говорит, что не так уж оно и must, имхо, запутывает окончательно.

интуитивно это воспринимается как:

зависитОбязательно: [
    {
        отСущности: 'такой-то',
        похуй: 'вальсируем' // но если вдруг, то порядок все-таки нужно поменять
    }
]

[qfox]

похуй: 'вальсируем'

Ох уж этот великий и могучий.