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

Remove deprecated ToxDNS #650

Merged
merged 1 commit into from Dec 29, 2017
Merged

Remove deprecated ToxDNS #650

merged 1 commit into from Dec 29, 2017

Conversation

@iphydf
Copy link
Member

@iphydf iphydf commented Dec 29, 2017

Based on #331.

Fixes #42.


This change is Reviewable

Based on #331.

Fixes #42.
@iphydf iphydf force-pushed the iphydf:rm-toxdns branch from 141526c to 2c8fb05 Dec 29, 2017
@robinlinden
Copy link
Member

@robinlinden robinlinden commented Dec 29, 2017

:lgtm_strong:


Reviewed 12 of 12 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@robinlinden robinlinden merged commit 2c8fb05 into TokTok:master Dec 29, 2017
4 of 5 checks passed
4 of 5 checks passed
ci/circleci CircleCI is running your tests
Details
code-review/reviewable 1/1 LGTMs
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details
@JFreegman
Copy link
Member

@JFreegman JFreegman commented Dec 29, 2017

:lgtm_strong:


Comments from Reviewable

@SkyzohKey
Copy link

@SkyzohKey SkyzohKey commented Dec 29, 2017

🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉

@iphydf iphydf deleted the iphydf:rm-toxdns branch Dec 29, 2017
@iphydf iphydf mentioned this pull request Dec 29, 2017
@iphydf iphydf added this to the v0.2.0 milestone Jan 6, 2018
@iphydf iphydf modified the milestones: v0.2.0-RC1, v0.2.0 Jan 14, 2018
@iphydf iphydf added refactor and removed code cleanup labels May 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.