Skip to content
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

Investigate how to remove cumbersome UMD js wrapper #14

Open
xzyfer opened this issue Feb 17, 2014 · 5 comments
Open

Investigate how to remove cumbersome UMD js wrapper #14

xzyfer opened this issue Feb 17, 2014 · 5 comments

Comments

@xzyfer
Copy link
Member

xzyfer commented Feb 17, 2014

The current js UMD wrapper is pretty gnarly. Ideally we could use AMD to manage internal dependencies but build js that's AMD free.

Down stream consumers can decide whether they want to shim our js or not. Forcing AMD on users is not ideal.

@geelen
Copy link
Contributor

geelen commented Feb 17, 2014

Will Browserify work? http://browserify.org/

I thought there was a way to make it output several different formats
(globals, AMD, etc), but maybe I'm getting confused. Someone else was
talking about writing your code with ES6 modules and using es6ify (
http://thlorenz.github.io/es6ify/) to compile them using traceur.

On 17 February 2014 17:58, xzyfer notifications@github.com wrote:

https://www.npmjs.org/package/amdclean

The current js UMD wrapper is pretty gnarly. Ideally we could use AMD to
manage internal dependencies but build js that's AMD free.

Down stream consumers can decide whether they want to shim our js or not.
Forcing AMD on users is not ideal.

Reply to this email directly or view it on GitHubhttps://github.com//issues/14
.

[image: Goodfilms]Glen MaddernCo-founderEmailglen@goodfil.msPhone0423 118
405Webhttp://goodfil.ms

@xzyfer
Copy link
Member Author

xzyfer commented Feb 17, 2014

I think it was me that suggested using ES6 modules, but at the time I'd read there were still some issues with transpiliers.

Both might be work looking into. If we can spitout valid AMD and plain old js from a single method that'd be awesome.

@xzyfer
Copy link
Member Author

xzyfer commented Feb 17, 2014

AMD clean is an option https://www.npmjs.org/package/amdclean

@xzyfer
Copy link
Member Author

xzyfer commented Feb 17, 2014

I've renamed the issue to be more accurate.

@gfranko
Copy link

gfranko commented Feb 19, 2014

If you guys need helping setting up AMDClean, let me know!

@xzyfer xzyfer added this to the 2.0.0 milestone Jul 28, 2014
@xzyfer xzyfer modified the milestone: 2.0.0 Jul 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants