Skip to content
This repository has been archived by the owner on Mar 25, 2018. It is now read-only.

Extend outgoing vinyl objects with BEM-related info #3

Open
qfox opened this issue Jul 19, 2016 · 8 comments
Open

Extend outgoing vinyl objects with BEM-related info #3

qfox opened this issue Jul 19, 2016 · 8 comments

Comments

@qfox
Copy link
Member

qfox commented Jul 19, 2016

Shortly, like that:

class BemVynil extends Vynil {
  entity: BemEntity,
  tech: String,
  level: String
}

This will be useful to build plugins like streamFilter(file => file.entity.block !== 'button').

We can add it as an option that will turn this logic on. Something like { bem: true } or more sensible ;-)

@qfox
Copy link
Member Author

qfox commented Aug 3, 2016

The latest vinyl version supports additional props in constructor: gulpjs/vinyl#92

@Yeti-or
Copy link
Member

Yeti-or commented Aug 3, 2016

what about data with evaled contents?

@qfox
Copy link
Member Author

qfox commented Aug 3, 2016

@Yeti-or what you want to eval? Browser.js? Or css?

@Yeti-or
Copy link
Member

Yeti-or commented Aug 3, 2016

I have eval and everything is seems eval`able to me now =(

@qfox
Copy link
Member Author

qfox commented Aug 3, 2016

eval my tra-la-la.

@blond
Copy link
Member

blond commented Aug 9, 2016

Something like { bem: true } or more sensible ;-)

Let's use BemVynil by defalut.

We can add bem option later.

@blond
Copy link
Member

blond commented Aug 9, 2016

what about data with evaled contents?

You can use custom props in Vinyl.

@qfox
Copy link
Member Author

qfox commented Aug 9, 2016

Let's use BemVynil by defalut.

Yup. Doing this right now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants