-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 zipfile for py2 #345
add zipfile for py2 #345
Conversation
class BadZipFile(Exception): ... | ||
|
||
if sys.version_info > (3,): | ||
BadZipfile = ... # type: Type[BadZipFile] |
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.
It might be easier to just say
BadZipfile = BadZipFile
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.
True, it's clearer.
0298144
to
14f290f
Compare
@@ -0,0 +1,94 @@ | |||
# Stubs for zipfile (Python 3.4) |
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.
Why have this comment here if this is supposed to be 2and3?
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.
Because typo, it from stubgen but I forgot to remove it, it is amended now.
Can you fix the failing 2.7 tests? |
Feel free to add "zipfile" to the pytype test blacklist for the time being, if that makes things easier. |
Opened pytype#28, added zipfile to blacklist for now. |
Remove py3 specific zipfile, rewrite it from doc.