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.

Sign up for GitHub

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

Jump to bottom

#53 require calling hook directly #54

Open

vojtatranta wants to merge 1 commit into css-modules:master from vojtatranta:53-fix-generateScopedName-not-called-issue

vojtatranta commented Jan 9, 2016

This is a draft how to fix this issue.
Unfortunately I don't understand well enough how require() works, so I need some help, thanks.

addresses #53


          require calling hook directly

6c48a74

Member

mightyaleksey commented Jan 11, 2016

Honestly, it's quite simple. I use require.extensions to make it work.

I'm not sure, that it will help you to fix the issue, but I'll try to think about it.

This import is resolved at compile time — that is, before the script starts executing. All the imports and exports of the declarative module dependency graph are resolved before execution.

from: http://calculist.org/blog/2012/06/29/static-module-resolution/

Thanks :)

mightyaleksey force-pushed the master branch from 255a105 to 985cc56 Compare

February 22, 2016 13:27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet