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

Rename data_replicator tests module #2547

Merged
merged 1 commit into from
Jan 24, 2019

Conversation

sakridge
Copy link
Member

Problem

The replicator name is associated with storage replicators, so
data_replicator sounds like that but it is actually a bunch of gossip
tests.

Summary of Changes

rename tests/data_replicator to tests/gossip

Fixes #

replicator name is associated with storage replicators, so
data_replicator sounds like that but it is actually a bunch of gossip
tests.
@codecov
Copy link

codecov bot commented Jan 24, 2019

Codecov Report

Merging #2547 into master will decrease coverage by 0.1%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #2547     +/-   ##
========================================
- Coverage    77.7%   77.6%   -0.2%     
========================================
  Files         110     110             
  Lines       18193   18171     -22     
========================================
- Hits        14148   14102     -46     
- Misses       4045    4069     +24

@sakridge sakridge merged commit 9845aec into solana-labs:master Jan 24, 2019
@sakridge sakridge deleted the rename-data-replicator branch January 24, 2019 23:50
AshwinSekar pushed a commit to AshwinSekar/solana that referenced this pull request Aug 14, 2024
* use governor rate limiter

* removed naive rate limter implementations

* clippy issue

* missing cargo.lock changes for programs crate

* panic on u64 to u32 overflow error

* safe check on u64 to u32 conversion
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