Add option to aggressively extract entities

bem-sdk-archive / bemjson-to-decl

BEMJSON to BEMDECL converter. DEPRECATED →

https://github.com/bem/bem-sdk/tree/master/packages/bemjson-to-decl

Other

2 stars 2 forks source link

Add option to aggressively extract entities #34

Closed smdenis closed 7 years ago

smdenis commented 7 years ago

Solves #29 cc @Yeti-or

Yeti-or commented 7 years ago

@creedencewright я понимаю @zxqfox предлагает включить эту опцию по умолчанию и вообще выбросить опцию

Yeti-or commented 7 years ago

и тесты ^__^

qfox commented 7 years ago

Так а опция то зачем?

Yeti-or commented 7 years ago

@zxqfox видимо чтобы нигде ниче не упало

coveralls commented 7 years ago

Coverage increased (+0.2%) to 97.368% when pulling b68bb538d00ed0ced71b57c17bf0f82c779ba150 on creedencewright:smd.aggressive into 6ec640000d3c00b765f161bbd59374410830e646 on bem-sdk:master.

qfox commented 7 years ago

@Yeti-or тесты же падают

/home/travis/build/bem-sdk/bemjson-to-decl/index.js
  59:13  error  Expected { after 'if' condition  curly

Yeti-or commented 7 years ago

@zxqfox да я в курсе, просто это форк, лениво было падал линтер — я пофиксил в отдельном pr