Skip to content

♻️ Handle MsgWithHeader in p2p module #1167

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 12, 2025
Merged

♻️ Handle MsgWithHeader in p2p module #1167

merged 1 commit into from
May 12, 2025

Conversation

wraitii
Copy link
Member

@wraitii wraitii commented May 9, 2025

No description provided.

@codecov Codecov
Copy link

codecov bot commented May 9, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 60.00000% with 32 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/p2p.rs 60.00% 32 Missing ⚠️
Files with missing lines Coverage Δ
src/consensus.rs 88.79% <ø> (ø)
src/mempool.rs 83.33% <ø> (ø)
src/p2p.rs 50.26% <60.00%> (ø)

@wraitii wraitii force-pushed the fixup_da_catchup branch from 838f26e to 205a177 Compare May 12, 2025 08:34
@wraitii wraitii force-pushed the handle_msg_with_header_p2p branch from 6bf8935 to 8c1c3af Compare May 12, 2025 08:35
Base automatically changed from fixup_da_catchup to process_bufferred May 12, 2025 08:55
@wraitii wraitii force-pushed the process_bufferred branch from 0aed9ea to 7406af4 Compare May 12, 2025 11:03
Base automatically changed from process_bufferred to main May 12, 2025 11:52
@wraitii wraitii force-pushed the handle_msg_with_header_p2p branch from 8c1c3af to b9cdb84 Compare May 12, 2025 11:56
@wraitii wraitii merged commit b3efc64 into main May 12, 2025
32 checks passed
@wraitii wraitii deleted the handle_msg_with_header_p2p branch May 12, 2025 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hhalex hhalex hhalex approved these changes

@maxgttph maxgttph maxgttph approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wraitii @hhalex @maxgttph