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

files.extract_file modifications #14

Closed
wants to merge 4 commits into from

Conversation

jteulade
Copy link
Contributor

@jteulade jteulade commented Apr 8, 2024

  • fix zip archive without any directory
  • create a sub directory in the case it doesn't exist in the zip
  • remove docker extract_files specific process -> can be slower
  • unify zip, tar and 7z archive extraction

Your checklist for this pull request

Please review the guidelines for contributing to this repository.

  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.

Description

Please describe your pull request.

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Thank you!

 * fix zip archive without any directory
 * create a sub directory in the case it doesn't exist in the zip
 * remove docker extract_files specific process -> can be slower
 * unify zip, tar and 7z archive extraction
@jteulade jteulade force-pushed the fix/extract_file_zip_without_directory branch from 2256ffc to bff04bd Compare April 8, 2024 14:28
@jteulade jteulade closed this Apr 8, 2024
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.

1 participant