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

feat(parse): add method to parse files #34

Closed qfox closed 8 years ago

qfox commented 8 years ago

Fixes #33

qfox commented 8 years ago

Seems like we need to get better name for it. Parse/stringify are not the best ones to do that.

/cc @blond @Yeti-or

blond commented 8 years ago

Need add tests should normalize decl for each BEMDECL version.

I think need use sinon.

qfox commented 8 years ago

Tests for normalize are out of scope of this PR.

sinon ok. For what purpose?

blond commented 8 years ago

Tests for normalize are out of scope of this PR. For what purpose?

Need check that normalize was called with the right options.

qfox commented 8 years ago

parse calls normalize and normalize-harmony internally with just one argument: decl: object[].

blond commented 8 years ago

parse calls normalize and normalize-harmony internally with just one argument: decl: object[].

Then check that calls necessary method with decl.

qfox commented 8 years ago

It is overkill, actually.

qfox commented 8 years ago

🆙

blond commented 8 years ago

👍