-
-
Notifications
You must be signed in to change notification settings - Fork 473
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Export locals if they are available and not on .use() #128
Conversation
+1 to this. Another way (if you don't want expose
|
@NekR I think you should have always access to locals, because that's the default behavior on others. My on my use case, I'm using React and created a React decorator that calls I had to change the style-loader by hand and include it on my project until this is merged. |
ping @sokra |
@cusspvz Yeah, I agree. I am currently using your solution too. Hope it'll be merged. |
@cusspvz we're having some issues regarding the CLA at the moment, it probably doesn't like that we moved this repo from webpack to webpack-contrib. You can sign the CLA here: https://cla.js.foundation/webpack/webpack. Sorry for the troubles. |
@SpaceK33z done! Yeah, I've also tried with |
@cusspvz - There were compliance issues we needed to clear up before the CLA bot could be enabled, that was cleared up about 10 days ago. To get the CLA signed, you just need to close this PR & open it again. That will trigger the CLA workflow again which should allow you to sign it. Please note, before you close & open it again ensure the email in your local git config matches what you have configured in github. |
Done! |
@cusspvz Thx 👍 |
style/useable
should match the usual expected behavior and export locals even if the styles aren't being used.On SSR,
use
will throw an error, and you will never be able to access locals.Please give priority to this. Thanks.