-
Notifications
You must be signed in to change notification settings - Fork 19
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
check pid in zip #105
check pid in zip #105
Conversation
/test |
1 similar comment
/test |
Successfully created a job for commit 6259dfd: |
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.
Can you please also add a test for writing-after-listing? To check the mode
part
/test |
Successfully created a job for commit 2c38b0c: |
/test |
Successfully created a job for commit 433cd75: |
chainerio/containers/zip.py
Outdated
@@ -80,12 +94,14 @@ def _wrap_fileobject(self, file_obj: Type['IOBase'], | |||
def open(self, file_path, mode='r', | |||
buffering=-1, encoding=None, errors=None, | |||
newline=None, closefd=True, opener=None): | |||
if sys.version_info < (3, 6) and "w" in mode: | |||
raise ValueError('Mode w and wb is not supported') |
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.
Can you mention that "w" and "wb" is not supported only in Python < 3.6?
/test |
Successfully created a job for commit 2ef8e26: |
/test |
Successfully created a job for commit 596fffe: |
/test |
Successfully created a job for commit a7ebe0e: |
This PR solves
ZipContainer
issue in #104