Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix expose AMD #19

Merged
merged 1 commit into from Mar 18, 2013

Conversation

Projects
None yet
2 participants
Contributor

eduardo-matos commented Mar 17, 2013

According to AMD API, you sould verify the existence of the amd property on the define function.

kriszyp added a commit that referenced this pull request Mar 18, 2013

@kriszyp kriszyp merged commit d60689d into kriszyp:master Mar 18, 2013

kriszyp added a commit that referenced this pull request Mar 19, 2013

Fixing the build by reverting the issue refs #19, as it breaks the Do…
…jo build since it doesn't properly use define.amd, fixes #20. Will fix #19 once Dojo build tool is fixed
Owner

kriszyp commented Mar 19, 2013

This should be reopened, but apparently you can't reopen an issue that is a pull request.

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