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

Add commonJS support #43

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add commonJS support #43

wants to merge 6 commits into from

Conversation

anzorb
Copy link

@anzorb anzorb commented Apr 8, 2014

Added commonJS module.exports support for Node/Browserify.

@mulderp
Copy link

mulderp commented Apr 15, 2014

+1

@traviswimer
Copy link

As far as I can tell, this plugin already worked with CommonJS. Apparently properties attached to this also get attached to module.exports.

On another topic, currently this will work well in a browser environment, but in NodeJS you will have to set global._ for this to work. It would be useful if this also had an option to include var _ = require("underscore"); in the generated template file. This is necessary if you run your unit tests with Node.

@flyfinec
Copy link

+1

2 similar comments
@willdady
Copy link

willdady commented Jul 7, 2014

+1

@SimonChong
Copy link

+1

Base automatically changed from master to main March 22, 2021 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants