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

Drop python 3.4 and 3.5 support #22

Merged
merged 1 commit into from
Oct 28, 2018
Merged

Drop python 3.4 and 3.5 support #22

merged 1 commit into from
Oct 28, 2018

Conversation

6ug
Copy link
Contributor

@6ug 6ug commented Oct 28, 2018

Part of #11 fixes, raised after #18 (comment)

Tested against code, works well!
screenshot from 2018-10-28 13-17-26

@6ug
Copy link
Contributor Author

6ug commented Oct 28, 2018

@carver @pipermerriam please review! 👍

Copy link
Collaborator

@carver carver left a comment

Choose a reason for hiding this comment

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

Looks good to me! I'll go ahead and merge. If @pipermerriam thinks of a reason we can't drop py3.5 support, then we can always add it back in before release.

setup.py Show resolved Hide resolved
@carver carver merged commit 25150ae into ethereum:master Oct 28, 2018
@6ug 6ug deleted the drop34and35 branch October 28, 2018 08:20
@6ug
Copy link
Contributor Author

6ug commented Oct 28, 2018

Thank you for merging! :) 💯

@pipermerriam
Copy link
Member

I think we need to keep 3.5 till py-evm drops it...

https://github.com/ethereum/py-evm/blob/9870841a126fb53a6cbab2ef7bf3d46262fa23a8/setup.py#L16

but we can just do this as a major version bump so lets just do that.

@hwwhww hwwhww mentioned this pull request Dec 11, 2018
pacrob pushed a commit to pacrob/py_ecc that referenced this pull request Oct 29, 2023
Remove attribution to specific person, update README
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.

None yet

3 participants