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

Use (exit 0) instead of true for windows install support. #79

Merged
merged 1 commit into from
Dec 12, 2011
Merged

Use (exit 0) instead of true for windows install support. #79

merged 1 commit into from
Dec 12, 2011

Conversation

chowey
Copy link
Contributor

@chowey chowey commented Dec 11, 2011

Fixes issue #72.

@chowey
Copy link
Contributor Author

chowey commented Dec 11, 2011

See also isaacs/npm#1625 which shows that this is the official way npm does it now.

@brianc
Copy link
Owner

brianc commented Dec 12, 2011

thanks @chowey

brianc added a commit that referenced this pull request Dec 12, 2011
Use `(exit 0)` instead of `true` for windows install support.
@brianc brianc merged commit d70a9bd into brianc:master Dec 12, 2011
@brianc
Copy link
Owner

brianc commented Dec 12, 2011

I'll do a version bump right after work

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