search

bem / webpack-bem-loader

Webpack BEM loader
Other
25 stars 14 forks source link

Rename to bem-loader #3

Closed awinogradov closed 7 years ago

awinogradov commented 8 years ago

So, I can republish (bem-loader)[https://www.npmjs.com/package/bem-loader] with the different name, and we'll use bem-loader.bem option in webpack.config instead bemLoader and bem in the list of loaders instead of webpack-bem.

Yeti-or commented 7 years ago

but right now this loader is almost only for bem-react

awinogradov commented 7 years ago

Definitely, no. It's for loading declaration and css for bem-entities. It's not about react in fact.

Yeti-or commented 7 years ago

okay not about react, but for loading declaraition we have no standart. right now we have only one implementation: applyDecls, and it's working only in bem-react-core

awinogradov commented 7 years ago

What about decls-loader? But we load CSS too :(

cc @veged @dfilatov

dfilatov commented 7 years ago

I don't quite undestand what the problem is, especially:

bem option in webpack.config instead bemLoader and bem in the list of loaders instead of webpack-bem

awinogradov commented 7 years ago

No problem with this option actually, because webpack2 hasn't this field and every loader should pass params by options field in the array of loaders near test, include and etc. This conversation about loader name. @Yeti-or want rename it to bem-react. I suggested bem-loader. Now its name is webpack-bem-loader.

veged commented 7 years ago

maybe just drop this question? ;-) cause with webpack2 it will be irrelevant

awinogradov commented 7 years ago

Yes, current loaders don't need name with special prefix