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

Comply with PEP 508 by using platform_python_implementation #5006

Merged
merged 1 commit into from
Oct 17, 2019

Conversation

seppeljordan
Copy link
Contributor

No description provided.

@seppeljordan
Copy link
Contributor Author

I will close this in a week, if no one takes care of this.

@alex
Copy link
Member

alex commented Oct 12, 2019

No need to close this. Both of pyca/cryptography's current maintainers are currently somewhat burned out on it, so we're signifantly slower to respond (and OpenSSL broke our tests so we can't straightforwardly merge things)

@seppeljordan
Copy link
Contributor Author

@alex Okay. Is there anything a regular user could do to help?

@alex
Copy link
Member

alex commented Oct 16, 2019

@seppeljordan Can you rebase/merge master please?

Also does this change the supported versions of pip/setuptools?

@reaperhulk reaperhulk closed this Oct 17, 2019
@reaperhulk reaperhulk reopened this Oct 17, 2019
Copy link
Member

@reaperhulk reaperhulk left a comment

Choose a reason for hiding this comment

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

Okay so our setuptools requirement is >=18.5 in setup.py but 40.5 in pyproject.toml. Looks like https://gsnedders.github.io/python-marker-test/results.html indicates we’re fine with this change for things that actually read pyproject.toml

@reaperhulk reaperhulk merged commit dc7f138 into pyca:master Oct 17, 2019
@seppeljordan seppeljordan deleted the pep-508-environment-marker branch October 17, 2019 09:04
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants