put the AMD definitions before CommonJS #22

Merged
merged 1 commit into from Jun 26, 2013

Conversation

Projects
None yet
2 participants
@phated
Contributor

phated commented Jun 18, 2013

Some build tools expose an exports object. UMDs that check for exports first will cause those build tools to error out.

jrburke added a commit that referenced this pull request Jun 26, 2013

Merge pull request #22 from phated/master
put the AMD definitions before CommonJS

@jrburke jrburke merged commit e2e09c0 into umdjs:master Jun 26, 2013

@jrburke

This comment has been minimized.

Show comment
Hide comment
@jrburke

jrburke Jun 26, 2013

Member

Merged, thank you very much!

Member

jrburke commented Jun 26, 2013

Merged, thank you very much!

@oxyc oxyc referenced this pull request in oxyc/umd-bestiejs Dec 24, 2013

Open

browserify-style build tool that wraps a given file #1

@ydaniv ydaniv referenced this pull request in julianshapiro/velocity Sep 8, 2014

Closed

require fix #288

jwngr pushed a commit to firebase/reactfire that referenced this pull request Oct 19, 2014

@Rich-Harris Rich-Harris referenced this pull request in esperantojs/esperanto Jan 9, 2015

Merged

Reshape UMD to respect module/export first. #42

@dashed dashed referenced this pull request in webpack/webpack Aug 22, 2015

Merged

define call should include empty dependencies list #1368

@jasonmit jasonmit referenced this pull request in moment/moment Sep 4, 2017

Closed

Enhancement: Attempt to use AMD before commonjs #4157

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