Skip to content

Only import enum34 on python versions < 3.4#4

Merged
abdullahselek merged 2 commits into
abdullahselek:masterfrom
eifinger:master
Jun 18, 2019
Merged

Only import enum34 on python versions < 3.4#4
abdullahselek merged 2 commits into
abdullahselek:masterfrom
eifinger:master

Conversation

@eifinger
Copy link
Copy Markdown
Contributor

@eifinger eifinger commented Jun 6, 2019

enum34 breaks python environments which are 3.6 or later.

With this PR it is only required when 3.4 or earlier is detected

@eifinger
Copy link
Copy Markdown
Contributor Author

Did you have a chance to take a look at this PR?

@abdullahselek abdullahselek self-requested a review June 18, 2019 15:27
@abdullahselek abdullahselek self-assigned this Jun 18, 2019
Copy link
Copy Markdown
Owner

@abdullahselek abdullahselek left a comment

Choose a reason for hiding this comment

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

Sorry for being late, looks good to me thanks for your contribution!

@abdullahselek abdullahselek merged commit 24be1c0 into abdullahselek:master Jun 18, 2019
@eifinger
Copy link
Copy Markdown
Contributor Author

Thanks!
Will you also release a new version on pypi?
My PR for home-assistant home-assistant/core#24603 is dependent on it ;-)

@abdullahselek
Copy link
Copy Markdown
Owner

Yes I will, but I’m in a holiday right now and don’t have my computer with me. Will let you know when I release a new version.

@eifinger
Copy link
Copy Markdown
Contributor Author

Thank you! Enjoy your holiday ;-)

@abdullahselek
Copy link
Copy Markdown
Owner

Hey @eifinger I just want to let you know that I've just release version 0.6.2.

@eifinger
Copy link
Copy Markdown
Contributor Author

Thank you!

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