-
Notifications
You must be signed in to change notification settings - Fork 26
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
add limited support for zip output (no encryption yet) #66
Conversation
Dear all, I replaced pyzipper by zipfile |
Codecov Report
@@ Coverage Diff @@
## main #66 +/- ##
==========================================
- Coverage 44.34% 44.07% -0.27%
==========================================
Files 22 23 +1
Lines 2783 2834 +51
==========================================
+ Hits 1234 1249 +15
- Misses 1549 1585 +36
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also run black
for the linting? Or tox -e fix
for running it with tox
.
Co-authored-by: Erik Schamper <[email protected]>
Co-authored-by: Erik Schamper <[email protected]>
…uire into feature/zip-output
Co-authored-by: Erik Schamper <[email protected]>
Hey @janstarke! First of all, thank for this contribution. It looks like the current implementation does not play nice with the acquisition of files in |
Co-authored-by: Stefan de Reuver <[email protected]>
Co-authored-by: Stefan de Reuver <[email protected]>
Co-authored-by: Stefan de Reuver <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small changes.
Co-authored-by: Erik Schamper <[email protected]>
Co-authored-by: Erik Schamper <[email protected]>
No description provided.