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

Bump leaky-bucket to 0.12.1 #16

Merged
merged 1 commit into from
Jul 10, 2022
Merged

Bump leaky-bucket to 0.12.1 #16

merged 1 commit into from
Jul 10, 2022

Conversation

udoprog
Copy link
Contributor

@udoprog udoprog commented Jul 10, 2022

Was looking over users of leaky-bucket and what's preventing people from upgrading. So I setup this PR to bump it for you (if you want).

Coordinator background task is no longer needed.

RateLimiter no longer implements Clone, but this was achieved internally before using an Arc to track interior state, so this is done externally instead.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 2645297879

  • 8 of 10 (80.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 86.667%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/rate_limiter.rs 7 9 77.78%
Totals Coverage Status
Change from base Build 2049499644: 0.4%
Covered Lines: 299
Relevant Lines: 345

💛 - Coveralls

@xnuter
Copy link
Owner

xnuter commented Jul 10, 2022

Thanks for upgrading LeakyBucket! 👍

@xnuter xnuter merged commit 0b8e5ac into xnuter:main Jul 10, 2022
@udoprog udoprog deleted the bump-rate-limiter branch July 11, 2022 05:53
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