Skip to content

chore(code): Remove HostMsg::ReceivedProposalPart message, use network subscription instead #943

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 1 commit into
base: main
Choose a base branch
from

Conversation

romac
Copy link
Member

@romac romac commented Mar 26, 2025

Closes: #XXX


PR author checklist

For all contributors

For external contributors

@romac romac force-pushed the romac/no-received-proposal-part-msg branch from 5f21547 to 79f1234 Compare March 26, 2025 10:30
Copy link

codecov bot commented Mar 26, 2025

❌ 33 Tests Failed:

Tests completed Failed Passed Skipped
155 33 122 0
View the top 3 failed test(s) by shortest run time
informalsystems-malachitebft-test::it n3f0_pubsub_protocol::broadcast_custom_config_2ktx
Stack Traces | 30s run time
No failure message available
informalsystems-malachitebft-test::it n3f0_pubsub_protocol::gossip_custom_config_1ktx
Stack Traces | 30s run time
No failure message available
informalsystems-malachitebft-test::it n3f1::proposer_fails_to_start
Stack Traces | 30s run time
No failure message available

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

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.

1 participant