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

feat: Add dht_get_nodes_response event to the events system. #2568

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

iphydf
Copy link
Member

@iphydf iphydf commented Jan 15, 2024

Should probably rename it to something starting with debug_ and make it opt-in. Right now, it absolutely spams the events object.


This change is Reviewable

@iphydf iphydf added this to the v0.2.19 milestone Jan 15, 2024
@iphydf iphydf force-pushed the dht-nodes-event branch 3 times, most recently from 008ac7e to d4aa455 Compare January 15, 2024 21:04
@iphydf iphydf marked this pull request as ready for review January 15, 2024 21:04
Copy link

codecov bot commented Jan 15, 2024

Codecov Report

Attention: 32 lines in your changes are missing coverage. Please review.

Comparison is base (e2c01e4) 66.90% compared to head (5b1f107) 66.91%.

❗ Current head 5b1f107 differs from pull request most recent head 16809dc. Consider uploading reports for the commit 16809dc to get more accurate results

Files Patch % Lines
toxcore/events/dht_get_nodes_response.c 73.49% 22 Missing ⚠️
toxcore/tox_dispatch.c 0.00% 5 Missing ⚠️
toxcore/tox_event.c 66.66% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2568      +/-   ##
==========================================
+ Coverage   66.90%   66.91%   +0.01%     
==========================================
  Files         147      148       +1     
  Lines       30177    30281     +104     
==========================================
+ Hits        20190    20264      +74     
- Misses       9987    10017      +30     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@iphydf iphydf force-pushed the dht-nodes-event branch 3 times, most recently from 825c042 to 5b1f107 Compare January 15, 2024 23:17
toxcore/events/dht_get_nodes_response.c Outdated Show resolved Hide resolved
toxcore/events/dht_get_nodes_response.c Outdated Show resolved Hide resolved
@pull-request-attention pull-request-attention bot assigned iphydf and robinlinden and unassigned iphydf Jan 16, 2024
Should probably rename it to something starting with `debug_` and make
it opt-in. Right now, it absolutely spams the events object.
@iphydf iphydf merged commit 16809dc into TokTok:master Jan 16, 2024
53 of 54 checks passed
@iphydf iphydf deleted the dht-nodes-event branch January 16, 2024 11:00
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.

2 participants