-
Notifications
You must be signed in to change notification settings - Fork 423
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
Google Closure module support #17
Comments
@beriberikix you're a little more familiar with the closure module system than I am (I think) :) Could you let me know if you think this addition is worth adding to a UMD (universal module def pattern). |
Only marginally. @cramforce did some work module server IIRC. |
@beriberikix Module server doesn't know anything about the closure module system actually and is really designed for CJS modules. |
I would generally favor a "needs multiple implementations" rule for UMD inclusion -- CJS, AMD, browser globals fit in this realm, but things like google closure, old Dojo and YUI would not. That said, I am not opposed to others working out different rules or different patterns. However, I will not have the time to maintain them or manage any pull requests for them. |
Support for the Google Closure module system is easy to add also.
Example: https://developers.google.com/closure/library/docs/tutorial
Just add this style at the top of the file:
The tool then used to build the full dependency tree is found here:
https://developers.google.com/closure/library/docs/depswriter
Regards
/ Fredrik Blomqvist
The text was updated successfully, but these errors were encountered: