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

Change nan dependency back to ^1.6.2 #161

Merged
merged 1 commit into from
Feb 6, 2015
Merged

Change nan dependency back to ^1.6.2 #161

merged 1 commit into from
Feb 6, 2015

Conversation

ide
Copy link
Contributor

@ide ide commented Feb 6, 2015

The issue with nan on io.js was fixed in nodejs/nan#273. npm install on io.js 1.1.0 works now.

The issue with nan on io.js was fixed in nodejs/nan#273. `npm install` on io.js 1.1.0 works now.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.36% when pulling 9881768 on ide:nan into 657d436 on lovell:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.36% when pulling 9881768 on ide:nan into 657d436 on lovell:master.

lovell added a commit that referenced this pull request Feb 6, 2015
Change nan dependency back to ^1.6.2
@lovell lovell merged commit 33880ce into lovell:master Feb 6, 2015
@lovell
Copy link
Owner

lovell commented Feb 6, 2015

Fantastic, thanks James.

@ide ide deleted the nan branch February 7, 2015 06:51
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.

3 participants