Skip to content

vote: update benches and tests to TowerSync#725

Merged
AshwinSekar merged 1 commit intoanza-xyz:masterfrom
AshwinSekar:tower-sync-tests
Apr 12, 2024
Merged

vote: update benches and tests to TowerSync#725
AshwinSekar merged 1 commit intoanza-xyz:masterfrom
AshwinSekar:tower-sync-tests

Conversation

@AshwinSekar
Copy link
Copy Markdown

Migrate remaining tests and benches to TowerSync

@mergify
Copy link
Copy Markdown

mergify Bot commented Apr 10, 2024

⚠️ The sha of the head commit of this PR conflicts with #584. Mergify cannot evaluate rules on this PR. ⚠️

@AshwinSekar AshwinSekar force-pushed the tower-sync-tests branch 2 times, most recently from aa845ed to 3b90c3f Compare April 11, 2024 19:54
Comment thread core/src/banking_stage.rs
VoteStateUpdate::from(vec![
(0, 8),
(1, 7),
(i as u64 + 64 + 5, 6),
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these were invalid before whoops

@AshwinSekar AshwinSekar marked this pull request as ready for review April 11, 2024 21:10
@AshwinSekar AshwinSekar requested a review from carllin April 11, 2024 21:10
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 98.80240% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 81.9%. Comparing base (ad94732) to head (a5fdb9a).

Additional details and impacted files
@@            Coverage Diff            @@
##           master     #725     +/-   ##
=========================================
- Coverage    81.9%    81.9%   -0.1%     
=========================================
  Files         851      851             
  Lines      231132   231226     +94     
=========================================
+ Hits       189484   189534     +50     
- Misses      41648    41692     +44     

@AshwinSekar AshwinSekar merged commit 499d36e into anza-xyz:master Apr 12, 2024
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.

3 participants