Skip to content

Conversation

@kidwm
Copy link

@kidwm kidwm commented Jan 1, 2016

Fix #2187 and tested manually.

@milesj
Copy link
Contributor

milesj commented Jan 5, 2016

I was about to give this a go but noticed you have already. However, this pull request could use some unit tests.

@facebook-github-bot
Copy link
Contributor

@kidwm updated the pull request.

@zpao
Copy link
Member

zpao commented Feb 26, 2016

Sorry for dropping the ball here. #6005 is more complete, so I'm going to work on getting that one merged. Thanks anyway though!

@zpao zpao closed this Feb 26, 2016
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.

4 participants