Skip to content

feat: WIP: new friend finding #1335

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

Draft
wants to merge 15 commits into
base: master
Choose a base branch
from
Draft

feat: WIP: new friend finding #1335

wants to merge 15 commits into from

Conversation

zugz
Copy link

@zugz zugz commented Aug 16, 2019

Implementing https://github.com/zugz/tox-DHTAnnouncements

For the purposes of the CLA, this PR is not a Contribution.


This change is Reviewable

@zugz zugz force-pushed the forwarding branch 2 times, most recently from 6e90943 to 3155508 Compare August 16, 2019 18:19
@zugz zugz closed this Aug 16, 2019
@zugz zugz reopened this Aug 16, 2019
@codecov
Copy link

codecov bot commented Aug 16, 2019

Codecov Report

Merging #1335 into master will decrease coverage by 0.1%.
The diff coverage is 96.9%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #1335     +/-   ##
========================================
- Coverage    84.6%   84.4%   -0.2%     
========================================
  Files          86      89      +3     
  Lines       15777   16053    +276     
========================================
+ Hits        13349   13551    +202     
- Misses       2428    2502     +74
Impacted Files Coverage Δ
toxcore/TCP_client.c 63.1% <100%> (-1.7%) ⬇️
toxcore/network.c 82% <100%> (+0.1%) ⬆️
toxcore/TCP_server.c 80.6% <100%> (-2.1%) ⬇️
toxcore/TCP_connection.c 78.7% <100%> (+1.2%) ⬆️
toxcore/net_crypto.c 94.7% <100%> (-0.3%) ⬇️
toxcore/forwarding.c 100% <100%> (ø)
toxcore/onion.c 98% <60%> (-0.8%) ⬇️
toxcore/onion_client.c 96% <80%> (-0.3%) ⬇️
toxcore/DHT.c 76.8% <88.4%> (-0.9%) ⬇️
auto_tests/forwarding_test.c 97.6% <97.6%> (ø)
... and 17 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ca370df...ed2b95e. Read the comment docs.

@zugz zugz force-pushed the forwarding branch 3 times, most recently from 40e7577 to ce75078 Compare August 16, 2019 19:31
@zugz zugz force-pushed the forwarding branch 4 times, most recently from 8d4a375 to ed2b95e Compare September 15, 2019 02:34
@iphydf iphydf added this to the v0.2.x milestone Mar 2, 2020
@TokTok TokTok deleted a comment from CLAassistant May 5, 2020
@iphydf iphydf changed the title WIP: new friend finding feat: WIP: new friend finding May 5, 2020
@auto-add-label auto-add-label bot added the enhancement New feature for the user, not a new feature for build script label May 5, 2020
@iphydf iphydf modified the milestones: v0.2.x, v0.3.0 Feb 4, 2022
@iphydf iphydf added P3 Low priority and removed P3 Low priority labels Feb 5, 2022
@iphydf iphydf marked this pull request as draft February 19, 2022 13:36
@zugz zugz force-pushed the forwarding branch 2 times, most recently from 8dc54df to 367bcb8 Compare March 9, 2022 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature for the user, not a new feature for build script
Development

Successfully merging this pull request may close these issues.

2 participants