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

some users need the AMD modules even if using npm install #102

Closed
millermedeiros opened this issue Nov 27, 2012 · 3 comments
Closed

some users need the AMD modules even if using npm install #102

millermedeiros opened this issue Nov 27, 2012 · 3 comments
Labels
Milestone

Comments

@millermedeiros
Copy link
Owner

for instance if they have test runners that should work on the browser and on node.js..

maybe use an UMD wrapper instead of converting source files into CJS or keep the src folder on the npm package.

@millermedeiros
Copy link
Owner Author

a solution if you want the AMD modules is to use bower to install the AMD version:

bower install amd-utils

@satazor
Copy link
Contributor

satazor commented Nov 27, 2012

One typical use case for this is a package that is built using amd but also has nodejs support, so it uses amdefine.
The package tests work on both node and in the browser but since amd-utils is now commonjs in npm, it won't work in the browser anymore.

@millermedeiros
Copy link
Owner Author

I pushed v0.9.0 to npm which includes the src folder, feel free to reopen in case that doesn't solve your issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants