Skip to content

feature: Zstandard compressor#551

Merged
WisdomPill merged 2 commits into
jazzband:masterfrom
vbudovski:feature/zstd-compressor
Oct 7, 2021
Merged

feature: Zstandard compressor#551
WisdomPill merged 2 commits into
jazzband:masterfrom
vbudovski:feature/zstd-compressor

Conversation

@vbudovski

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Sep 26, 2021

Copy link
Copy Markdown

Codecov Report

Merging #551 (2678c05) into master (827eb4e) will increase coverage by 0.2%.
The diff coverage is 100.0%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #551     +/-   ##
========================================
+ Coverage    83.4%   83.6%   +0.2%     
========================================
  Files          20      21      +1     
  Lines        1157    1171     +14     
  Branches       60      60             
========================================
+ Hits          964     978     +14     
  Misses        187     187             
  Partials        6       6             
Flag Coverage Δ
mypy 33.3% <61.6%> (+0.4%) ⬆️
tests 82.7% <100.0%> (+0.3%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
django_redis/compressors/zstd.py 100.0% <100.0%> (ø)

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 827eb4e...2678c05. Read the comment docs.

@WisdomPill

WisdomPill commented Sep 26, 2021

Copy link
Copy Markdown
Member

thanks for the PR @vbudovski, just the mypy error is needed to be solved, redis-master is broken for a while now, I will have a look at that and solve it in another issue

@WisdomPill WisdomPill merged commit 80f791b into jazzband:master Oct 7, 2021
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