Skip to content

Conversation

@bonsaiviking
Copy link

No description provided.

@travisbot
Copy link

This pull request passes (merged 06fdd7a into 4c6aa6e).

@weppos
Copy link
Collaborator

weppos commented May 8, 2013

Hello @bonsaiviking,

Thanks for your patch. In order to merge the patch, each change must be validated with an unit test. Would you mind to update the patch?

-- Simone

@bonsaiviking
Copy link
Author

@weppos I've added the unit tests and rebased against master.

@ghost
Copy link

ghost commented May 12, 2014

I have forked this repository (after attempting and failing to contact author) here: https://github.com/mordocai/net-dns. If you wish to resubmit your pull request there I will review/merge it in. It will shortly be the gem net-dns2 on rubygems.org.

@ghost
Copy link

ghost commented May 13, 2014

I have merged your changes (minus tests, i'm writing new rspec tests) into my fork here: https://github.com/mordocai/net-dns/commit/81391af458a033d6a9bff4233c833b7364689cb4. It should be in the v0.8.3 release later this week.

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