enb / enb-bem-examples

BEM examples for ENB
Other
12 stars 15 forks source link

Поменял логику работы pseudo-levels builder #27

Closed eGavr closed 8 years ago

eGavr commented 8 years ago

/cc @blond

closes #15 #18

Не подключается уровень *.examples/blocks если собирать пример точечно
$ enb make examples desktop.examples/block/example # не подключается
$ enb make examples desktop.examples/block # подключается

Ожидается, что уровень *.examples/blocksбудет подключаться для любого примера, вне зависимости от того как мы его собираем.


Решение

Папка-симлинка с общими блоками примеров блока создается в каждом примере отдельно, то есть было:

 <set-name>.examples
  └── <block-name>/
       ├── .blocks/             # уровень для всех примеров блока <block-name>
       ├── 10-simple/
           ├── blocks/          # уровень для примера 10-simple
           └── 10-simple.bemjson.js.symlink
       └── 20-complex/
           └── 20-comples.bemjson.js.symlink

а стало:

 <set-name>.examples
  └── <block-name>/
       ├── 10-simple/
           ├── <block-name>.blocks/    # уровень для всех примеров блока <block-name>
           ├── 10-simple.blocks/       # уровень для примера 10-simple
           └── 10-simple.bemjson.js.symlink
       └── 20-complex/
           ├── <block-name>.blocks/    # уровень для всех примеров блока <block-name>
           └── 20-comples.bemjson.js.symlink

Важно! Мажорное изменение!

qfox commented 8 years ago

Кажется, что стало лучше, но плохо что нет тестов ;-(

blond commented 8 years ago

Кажется, что стало лучше, но плохо что нет тестов ;-(

Проверяем запуском на реальных проектах.

blond commented 8 years ago

@eGavr, шикарненько! ))

LGTM