Added "node" compatibility #2

Merged
merged 1 commit into from Nov 6, 2011

Projects

None yet

3 participants

@shinuza
shinuza commented Nov 6, 2011

I figured this would be a nice addition at almost no cost (in terms of code length).

Feel free to use this and create a package.json, i.e for npm registration.

@addyosmani
Owner

This looks good. Thanks for the pull!

@addyosmani addyosmani merged commit 5e0f97e into addyosmani:master Nov 6, 2011
@dotnetCarpenter

root.templatez is not defined if exports exist, right?

Right, could this be an issue?

Owner

@shinuza It could be. We might need to back this out until we can figure out a better solution.

This bit was inspired by underscore/backbone. You might want to check it out, there's also work to make it compatible with AMD, it depends how far you want to take it.

http://documentcloud.github.com/underscore/docs/underscore.html

Owner

So that would be :

-Simple version
-Minified version
-AMD/CommonJS Modules

Sounds good to me, now you may want to have a build system to create these on the fly.

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