search

michalkvasnicak / babel-plugin-css-modules-transform

Extract css class names from required css module files, so we can render it on server.
MIT License
326 stars 54 forks source link

Problems on Windows machines #110

Closed birdofpreyru closed 4 years ago

birdofpreyru commented 4 years ago

Because of https://github.com/css-modules/css-modules-require-hook/issues/119, babel-plugin-css-modules-transform does not work on Windows machines either (generates hashes different from those generated by css-loader). Part of the problem is that owner of css-modules-require-hook is not responsive (the issue and correct fix are pending for one year there). So, I am considering to fork and fix css-modules-require-hook, and wanna check with you @michalkvasnicak whether you are open to update this package to depend on my fixed version of css-module-require-hook?

michalkvasnicak commented 4 years ago

@birdofpreyru Hello I'm sorry I wasn't really invest my time into this package. I want to rewrite it to typescript and with babel 7 support only. I don't see any problem in supporting your library. Feel free to open PR for 1.x version to use your package.

birdofpreyru commented 4 years ago

Nevermind, I figured out I better fork your lib and give it a full revamp in my taste. If anybody needs it, I published my version here: https://www.npmjs.com/package/@dr.pogodin/babel-plugin-css-modules-transform