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

Test cryptography 3.0 on Python 3 #218

Closed
wants to merge 1 commit into from

Conversation

pquentin
Copy link
Member

cryptography 3.0 does support Python 2, but issues a deprecation warning at import that I wasn't able to ignore: pyca/cryptography#5346.

@webknjaz
Copy link
Member

Ref: pyca/cryptography#5346 (comment).

@pquentin
Copy link
Member Author

Thanks, your comment there fixed it. I guess that's an easier option as it will take care of the Windows issue too.

@pquentin pquentin mentioned this pull request Jul 27, 2020
@webknjaz
Copy link
Member

Give me a few min and I'll send a pytest config that would be a better place for all those options...

@webknjaz
Copy link
Member

This is what I usually have in other projects: #220. (not setup.cfg because flat is better than nested and namespaces are honking great idea -- it's easier to spot vs having to guess in what file it is, among setup.cfg, tox, pyproject etc)

@webknjaz
Copy link
Member

I've also added #221 as a follow-up for #220. You should probably merge them before proceeding here.

@pquentin pquentin closed this Jul 29, 2020
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