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

[doc] Pip Cryptography Install Version #2549

Closed
wants to merge 1 commit into from

Conversation

jasondscott
Copy link

Description:
Adding a specific version to the pip install intructions for
cryptography.

Why:
So that it matches the required version in setup.py.
Without this, the latest version is installed with the correct flags and
then when setup.py is ran 1.7.2 is installed with no flags which will
cause other errors.

Changes:
Added specific verion to pip install.

Description:
Adding a specific version to the pip install intructions for
cryptography.

Why:
So that it matches the required version in setup.py.
Without this, the latest version is installed with the correct flags and
then when setup.py is ran 1.7.2 is installed with no flags which will
cause other errors.

Changes:
Added specific verion to pip install.
@mistercrunch
Copy link
Member

I wonder if we should just bump the version to the latest in setup.py ?

@jasondscott
Copy link
Author

@mistercrunch It is generally good to stay up to date with the latest with security packages so I would be in favor of bumping the version to the latest. I'm not sure the process that you use to test/validate a new package version though. Let me know if I can help.

I thought this would be a less controversial change than bumping the version.

@mistercrunch
Copy link
Member

I'll do a round of bumping now. This service makes it straightforward:
https://requires.io/github/airbnb/superset/requirements/?branch=master

@mistercrunch
Copy link
Member

#2575

@mistercrunch
Copy link
Member

I bumped the version to the latest.

@jasondscott
Copy link
Author

👍 Thanks @mistercrunch

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