bem-sdk-archive / bem-decl

:capital_abcd: Manage declaration of BEM entities. DEPRECATED →
https://github.com/bem/bem-sdk/tree/master/packages/decl
Other
11 stars 3 forks source link

Add normalize2 stub and tests #41

Closed skad0 closed 8 years ago

skad0 commented 8 years ago

Test to understand and format required format

blond commented 8 years ago

Хорошо бы подумать про обработку невалидных кейсов, которые, не указаны в SchemeJSON.

qfox commented 8 years ago

bemDecl.normalizer('normalize2')({elem: 'control'}) выдает [ { block: undefined, elem: 'q' } ] Это прекрасно, но не очень.

Теоретически можно после нормализации делать .map(item => fulfill(item, scope)) Или даже .map(fulfill.bind(this, scope)), надо будет только в fulfill скоуп поставить на первое место.

Даже если оно сейчас будет работать после fulfill правильно — block: undefined выглядит не очень понятно.

qfox commented 8 years ago

Хорошо бы подумать про обработку невалидных кейсов, которые, не указаны в SchemeJSON.

https://github.com/bem-sdk/bem-decl/issues/45

qfox commented 8 years ago

@Yeti-or @tadatuta @skad0 Я готов вливать ;-) Всё норм?

skad0 commented 8 years ago

@zxqfox да, проблем не вижу, спасибо

tadatuta commented 8 years ago

lgtm

Yeti-or commented 8 years ago

🎱