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

NPM package file and a couple of fixes #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kaievns
Copy link

@kaievns kaievns commented Mar 16, 2011

Hi Chris,

I've made a simple npm package so people could install and use envjs with npm.

And I also did couple of fixes to make it run on node.js (not sure if I've done it the right way though).

Cheers,
Nik

@thatcher
Copy link
Owner

This is awesome, thank you! I'll try to get it integrated asap, but it might take a couple days.

@kaievns
Copy link
Author

kaievns commented Mar 16, 2011

sure, no hurry with that, anyone can install it locally without actually having it on the NPM server.

btw, there is no window or document variables in node when you include the envjs/node.js file, I currently create them manually via var window = new Window(). Well, just letting you know, not sure what's the plan.

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

2 participants