Skip to content

Conversation

@miurahr
Copy link
Owner

@miurahr miurahr commented Jan 29, 2020

COPY method is defined to use make 7-zip archive from 7-zip or zip archive to avoid a duplicated compression.

Copy link
Owner Author

@miurahr miurahr left a comment

Choose a reason for hiding this comment

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

Need to fix lint error

@codecov
Copy link

codecov bot commented Jan 29, 2020

Codecov Report

Merging #61 into master will decrease coverage by 0.04%.
The diff coverage is 78.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage   83.71%   83.67%   -0.05%     
==========================================
  Files           8        8              
  Lines        2156     2169      +13     
  Branches      352      352              
==========================================
+ Hits         1805     1815      +10     
- Misses        225      228       +3     
  Partials      126      126
Flag Coverage Δ
#linux 82.75% <78.33%> (-0.04%) ⬇️
#mac 82.57% <78.33%> (-0.04%) ⬇️
#py35 79.32% <78.33%> (-0.02%) ⬇️
#py36 83.34% <78.33%> (-0.04%) ⬇️
#py37 83.34% <78.33%> (-0.04%) ⬇️
#py38 83.47% <78.33%> (-0.04%) ⬇️
#windows 82.43% <78.33%> (-0.04%) ⬇️
Impacted Files Coverage Δ
py7zr/compression.py 79.81% <78.33%> (+0.5%) ⬆️
py7zr/archiveinfo.py 83.17% <0%> (-0.24%) ⬇️

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 6ba709c...18a1a6a. Read the comment docs.

COPY method is defined to use make 7-zip archive from 7-zip or zip archive to avoid a duplicated compression.
@miurahr miurahr force-pushed the topic-copy-decompressor branch from 5ca6664 to 18a1a6a Compare January 29, 2020 12:26
@miurahr miurahr merged commit 3319938 into master Jan 29, 2020
@miurahr miurahr deleted the topic-copy-decompressor branch January 29, 2020 15:11
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