Refactored walk arguments
#30
Conversation
|
Типо теперь можно будет сканить не олько уровни, но и конкретные блоки? |
|
@eGavr, нет :) Но это шаг к этому. |
|
Я не понимаю, почему не научить bem-walk сканировать не только уровни, а и любую сущность в принципе? Если мне нужен будет bem-walk на еще нескольких пакетах, то снова реализовывать эту логику, так может вынести ее в сам bem-walk? Ты когда-то давно говорил, что таким способом мы потеряем в скорости... И че? В общем, ИМХО, для bem-walk это будет не лишним :
|
|
@eGavr а как же делать только одну вещь, но делать её хорошо? 2015-03-28 9:07 GMT+03:00 Evgeniy Gavryushin notifications@github.com:
|
|
@eGavr, про частичное сканирование есть задача — #14. От неё никто не отказывался ;) Проблема в том, задача сложная, и за один день её не запилить.
Это заведомо ложное утверждение =) Что касается скорости —это очень важная фича, которая заявлена изначально. Терять её для всех, ради задачи не для всех — неправильно. По поводу разных фич: не хочется превращать тулзу в убер-штуку, которая умеет абсолютно всё. Политика развития в том, что бы инструмент умел общие (читай как common, не merged) для всех вещи, но позволял у себя реализовать любые выкрутасы. Если ты считаешь, что какие-то задачи можно решить проще — заводи ишью по каждому вопросу, будем обсуждать ;)
Что ты имеешь ввиду? Если ты о данных каждого встретившегося файла, от он такой — #12. Но если добавлять другие сущности/события валкера, то у них формат может быть другой, что вроде логично — #20.
Для каждой схемы, это довольно индивидуальное поведение. Скажем для flex или Кроме того, способ ругаться тоже не очень понятен: варнинги, ошибки, события? Если есть предложить что-то конкретное — заводи ишью, обсудим ;) |
ээээ... bem-walk - это сканер бэм сущностей по задумке, я так понимаю, поэтому добавление функционала ПРО СКАНИРОВАНИЕ СУЩНОСТЕЙ не превратит тулзу в убер штуку, а улучшит ее для пользователя.
упустил этот момент - согласен, что стандарт есть, тогда если он есть , то что стопит выпустить версию?
Пожалуй стоит просто давать какую-то инфу, что файл найден и не пропускать его, например, имени и полного пути было бы достаточно, ну и какое-то отдельное поле , которое бы говорило, что это не БЭМ сущность. |
**1. The second argument now expects `config` instead of `options`.**
Options by default now need specify in `config.defaults` instead of
`options`.
*Before:*
```js
walk(['blocks'], { sheme: 'flat' });
```
*After:*
```js
var config = {
defaults: { sheme: 'flat' }
};
walk(['blocks'], config);
```
**2. Info about levels (scheme and naming) now need specify in
`config.levels` instead of `levels` arguments.**
The `levels` argument as `Array` no longer supported.
*Before:*
```js
walk([{ path: 'blocks', sheme: 'flat' }]);
```
*After:*
```js
var config = {
levels: {
blocks: { sheme: 'flat' }
}
};
walk(['blocks'], config);
```
Resolved #15
1. The second argument now expects
configinstead ofoptions.Options by default now should be specified in
config.defaultsinstead ofoptions.Before:
After:
2. Info about levels (scheme and naming) now need specify in
config.levelsinstead oflevelsarguments.The
levelsargument asArrayis no longer supported.Before:
After: