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

chore(sequencing): remove ConsensusMessage and Proposal from proto #2559

Open
wants to merge 1 commit into
base: guyn/streams/remove_consensus_message4
Choose a base branch
from

Conversation

guy-starkware
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch 2 times, most recently from 330c149 to 8860f65 Compare December 9, 2024 11:12
Copy link

codecov bot commented Dec 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (guyn/streams/remove_consensus_message4@2a8efb4). Learn more about missing BASE report.

Additional details and impacted files
@@                            Coverage Diff                            @@
##             guyn/streams/remove_consensus_message4    #2559   +/-   ##
=========================================================================
  Coverage                                          ?   35.49%           
=========================================================================
  Files                                             ?       51           
  Lines                                             ?     6255           
  Branches                                          ?     6255           
=========================================================================
  Hits                                              ?     2220           
  Misses                                            ?     3742           
  Partials                                          ?      293           

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

@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from 211056f to c9ddaa9 Compare December 9, 2024 13:24
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from 8860f65 to 5cfab82 Compare December 9, 2024 13:24
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from c9ddaa9 to 3ee8a71 Compare December 10, 2024 07:32
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from 5cfab82 to 6d436fa Compare December 10, 2024 07:32
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from 3ee8a71 to c0376a7 Compare December 10, 2024 07:41
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from 6d436fa to dfcfa5e Compare December 10, 2024 07:41
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from c0376a7 to 969962d Compare December 10, 2024 08:11
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from dfcfa5e to 17bb2e4 Compare December 10, 2024 08:11
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from 969962d to c0021ad Compare December 10, 2024 08:26
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from 17bb2e4 to 24fa22b Compare December 10, 2024 08:26
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from c0021ad to b5f3e18 Compare December 10, 2024 09:53
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from 24fa22b to e3e97d9 Compare December 10, 2024 09:54
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from b5f3e18 to de5f157 Compare December 10, 2024 10:33
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from e3e97d9 to bc60ec0 Compare December 10, 2024 10:33
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from de5f157 to b8f55f5 Compare December 10, 2024 11:01
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from bc60ec0 to f75035f Compare December 10, 2024 11:01
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from b8f55f5 to b68a449 Compare December 10, 2024 13:11
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from f75035f to bb60bc9 Compare December 10, 2024 13:11
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from b68a449 to 8d5c718 Compare December 10, 2024 17:02
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from bb60bc9 to c3e5da4 Compare December 10, 2024 17:03
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from 8d5c718 to e1454b3 Compare December 11, 2024 16:52
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from c3e5da4 to eb76a4d Compare December 11, 2024 16:52
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from e1454b3 to 2a8efb4 Compare December 11, 2024 19:32
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from eb76a4d to f5fe2a7 Compare December 11, 2024 19:32
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from 2a8efb4 to 14b0b01 Compare December 15, 2024 11:06
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from f5fe2a7 to e9559ee Compare December 15, 2024 11:06
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message4 branch from 14b0b01 to c667343 Compare December 19, 2024 19:10
@guy-starkware guy-starkware force-pushed the guyn/streams/remove_consensus_message_proposal_from_proto branch from e9559ee to b67d1bb Compare December 19, 2024 19:10
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