FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Remove unnecessary compatibility shim for BytesIO by jdufresne · Pull Request #401 · html5lib/html5lib-python · GitHub

Remove unnecessary compatibility shim for BytesIO - #401

Merged
gsnedders merged 1 commit into
html5lib:masterfrom
jdufresne:bytesio
Oct 1, 2018
Merged

Remove unnecessary compatibility shim for BytesIO#401
gsnedders merged 1 commit into
html5lib:masterfrom
jdufresne:bytesio

Conversation

Copy link
Copy Markdown
Contributor

io.BytesIO is available on all supported Pythons. Can simplify by assuming the import will succeed.

io.BytesIO is available on all supported Pythons. Can simplify by
assuming the import will succeed.

Copy link
Copy Markdown
Contributor Author

The AppVeyor failures are unrelated to this changed and are fixed in #397.

gsnedders left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Is this not supported from 2.6? I'm not sure we've ever supported < 2.6 and 3, so I dunno why that's even there. Oh well! Thanks for catching this.

gsnedders merged commit 0ba6628 into html5lib:master Oct 1, 2018
jdufresne deleted the bytesio branch February 26, 2020 01:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL