vutran1710 / PyrateLimiter

⚔️Python Rate-Limiter using Leaky-Bucket Algorithm Family
https://pyratelimiter.readthedocs.io
MIT License
349 stars 36 forks source link

Improvements for local dev and CI tools #66

Closed JWCook closed 2 years ago

JWCook commented 2 years ago

Closes #65. This also adds some other pytest, coverage, and GitHub Actions features that I use in my own projects. If there are any of these items you don't want, let me know and I'll remove it from the PR.

Scripts

GitHub Actions

codecov[bot] commented 2 years ago

Codecov Report

Merging #66 (a8b9a45) into master (2e68f71) will decrease coverage by 0.90%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
- Coverage   98.73%   97.82%   -0.91%     
==========================================
  Files           8        8              
  Lines         316      322       +6     
  Branches        0       30      +30     
==========================================
+ Hits          312      315       +3     
  Misses          4        4              
- Partials        0        3       +3     
Impacted Files Coverage Δ
pyrate_limiter/bucket.py 93.51% <0.00%> (-2.78%) :arrow_down:
pyrate_limiter/request_rate.py 100.00% <0.00%> (ø)
pyrate_limiter/limit_context_decorator.py 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1087514...a8b9a45. Read the comment docs.

JWCook commented 2 years ago

I'm not quite sure why that Codedov report is saying coverage decreased.

vutran1710 commented 2 years ago

I'm not quite sure why that Codedov report is saying coverage decreased.

i dont think its a big deal. if you think evrything ready, i will merge the pr.

JWCook commented 2 years ago

Yes, if it all looks good to you, go ahead and merge it.