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

Fix url #45

Merged
merged 1 commit into from
Jun 1, 2015
Merged

Fix url #45

merged 1 commit into from
Jun 1, 2015

Conversation

kevinsimper
Copy link
Contributor

The url was wrong, so if you tried to go from npmjs.com then 404

@iteles
Copy link
Member

iteles commented May 31, 2015

Hey @kevinsimper, thanks for submitting this PR!
Are you able to make the changes required so that the Travis build passes?

@kevinsimper
Copy link
Contributor Author

It is not something I can fix, it is an error in io.js.

nodejs/node#1850

@nelsonic
Copy link
Member

nelsonic commented Jun 1, 2015

Looks like the fix has been applied to io.js but might not be on Travis-CI yet...

@nelsonic
Copy link
Member

nelsonic commented Jun 1, 2015

I'm satisfied that this is working on Node.js v.0.10 & v0.12 which are the ones _most people_ are using. @iteles lets merge this! thanks!

nelsonic added a commit that referenced this pull request Jun 1, 2015
@nelsonic nelsonic merged commit 01d8da5 into dwyl:master Jun 1, 2015
@kevinsimper kevinsimper deleted the patch-1 branch June 1, 2015 08:55
@nelsonic
Copy link
Member

nelsonic commented Jun 1, 2015

thanks @kevinsimper 👍

@iteles
Copy link
Member

iteles commented Jun 1, 2015

👍 Thanks!

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

Successfully merging this pull request may close these issues.

3 participants