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

Fixes for Python 3.6 support #957

Merged
merged 3 commits into from
Aug 9, 2019

Conversation

andrewwhitehead
Copy link
Member

  • Prefer hashlib.sha3_256 when it's available (in python 3.6+) - the sha3 module is not compatible

  • Fix a type checking error that occurs in python 3.6+

This is an update of PR #714

Signed-off-by: Andrew Whitehead <andrew@1crm.com>
Signed-off-by: Andrew Whitehead <andrew@1crm.com>
@evernym-ci
Copy link
Contributor

Can one of the admins verify this patch?

@ghost
Copy link

ghost commented Oct 26, 2018

Could one of the committers please verify this patch?

@ashcherbakov
Copy link
Contributor

We need Python 3.6 CI/CD environment to make sure that all tests pass on Python 3.6

@ashcherbakov ashcherbakov reopened this Nov 7, 2018
Signed-off-by: Andrew Whitehead <andrew@1crm.com>
@andrewwhitehead
Copy link
Member Author

@ashcherbakov Is there any timeline for adding a Python 3.6 testing environment? Without this change the library doesn't run at all on 3.6 and it's becoming increasingly difficult for us to support 3.5 (in von-network).

@ashcherbakov
Copy link
Contributor

@CyWolf We need to check with our Product, but I think it will be at the time of Ubuntu 18.04 support, which is scheduled at the end of this year - beginning of next year.

@andrewwhitehead
Copy link
Member Author

Note: PR #974 includes these changes as well as Python 3.7 compatibility.

@ashcherbakov
Copy link
Contributor

(ci) test this please

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