Skip to content

Conversation

bnoordhuis
Copy link
Member

@bnoordhuis
Copy link
Member Author

Not much to review, I guess.

@saghul
Copy link
Member

saghul commented Feb 26, 2015

Not really, indeed. I hope the CI is happy now :-)

@indutny
Copy link
Member

indutny commented Feb 26, 2015

@bnoordhuis should we land it?

@jbergstroem
Copy link
Member

Fixes failing tests on OS X 10.10 for me.

@bnoordhuis
Copy link
Member Author

@indutny Can I get a LGTM?

@indutny
Copy link
Member

indutny commented Feb 26, 2015

If you'll ask kindly :) LGTM

PR-URL: nodejs#966
Reviewed-By: Fedor Indutny <[email protected]>
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.

4 participants