-
Notifications
You must be signed in to change notification settings - Fork 100
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
feat(routing/dht/zikade): add dht routing implementation using zikade #472
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
routing/dht
does not exists btw, no need to add v2
Understood. I chose this on the assumption that dht v1 would still be needed for a while and might end up in Boxo. Happy to remove the v2 if needed |
@iand : just in case this was an accident, the team teans to not look at draft PRs. Please callout if you're wanting feedback on the current state (or move out of draft). |
@BigLep it's not an accident, I wanted early visibility while I am working on it. Will mark it ready for review when complete. |
Codecov Report
@@ Coverage Diff @@
## main #472 +/- ##
==========================================
- Coverage 65.57% 65.51% -0.06%
==========================================
Files 207 207
Lines 25543 25543
==========================================
- Hits 16749 16735 -14
- Misses 7323 7335 +12
- Partials 1471 1473 +2 |
@Jorropo would you be able to take another look? |
Add support for the Zikade Amino DHT implementation as a routing choice.