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

Make KademliaProtocol public #1135

Merged
merged 2 commits into from
Dec 22, 2020
Merged

Conversation

limebell
Copy link
Member

The change should have been included in #1120, but omitted by mistake.

riemannulus
riemannulus previously approved these changes Dec 22, 2020
moreal
moreal previously approved these changes Dec 22, 2020
@limebell limebell dismissed stale reviews from moreal and riemannulus via 13a742c December 22, 2020 05:13
@codecov
Copy link

codecov bot commented Dec 22, 2020

Codecov Report

Merging #1135 (08c3a18) into main (64b0d03) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1135      +/-   ##
==========================================
- Coverage   87.63%   87.59%   -0.05%     
==========================================
  Files         345      345              
  Lines       29956    29956              
==========================================
- Hits        26253    26239      -14     
- Misses       2073     2087      +14     
  Partials     1630     1630              
Impacted Files Coverage Δ
Libplanet/Net/Protocols/KademliaProtocol.cs 78.61% <ø> (-0.40%) ⬇️
Libplanet/Net/Protocols/KBucket.cs 91.52% <0.00%> (-2.55%) ⬇️
Libplanet.Stun/Stun/TurnClient.cs 78.23% <0.00%> (-1.37%) ⬇️
Libplanet/Net/Swarm.cs 82.11% <0.00%> (-0.96%) ⬇️
Libplanet/Net/NetMQTransport.cs 79.41% <0.00%> (+1.08%) ⬆️

dahlia
dahlia previously approved these changes Dec 22, 2020
moreal
moreal previously approved these changes Dec 22, 2020
limebell and others added 2 commits December 22, 2020 15:17
Co-authored-by: Hong Minhee (洪 民憙) <hong.minhee@gmail.com>
@limebell limebell merged commit cb19702 into planetarium:main Dec 22, 2020
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.

5 participants