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

Specify supported version(s) of Python in setup.py #378

Merged
merged 1 commit into from
Mar 2, 2019

Conversation

Freso
Copy link
Member

@Freso Freso commented Mar 1, 2019

In preparation for dropping Python 2.7 support, this specifies versions that are compatible with Python 2.7, so anyone installing via pip (if we end up publishing to PyPI) will get the proper version.

See https://packaging.python.org/guides/dropping-older-python-versions/

@Freso Freso added the Improvement Minor improvement to code label Mar 1, 2019
@Freso Freso added this to the 0.8.0 milestone Mar 1, 2019
@Freso Freso self-assigned this Mar 1, 2019
@Freso Freso requested a review from JoeLametta March 1, 2019 10:11
In preparation for dropping Python 2.7 support[1], this specifies
versions that *are* compatible with Python 2.7, so anyone installing via
pip (if we end up publishing to PyPI) will get the proper version.

See https://packaging.python.org/guides/dropping-older-python-versions/

[1] #78

Signed-off-by: Frederik “Freso” S. Olesen <freso.dk@gmail.com>
@MerlijnWajer
Copy link
Collaborator

Looks good to me.

@Freso Freso merged commit a9bb51a into develop Mar 2, 2019
@Freso Freso deleted the specify-python-support branch March 2, 2019 10:21
@Freso
Copy link
Member Author

Freso commented Mar 2, 2019

@MerlijnWajer gave the go ahead to pull this into "develop" on IRC, so went ahead and did just that.

pombredanne added a commit to aboutcode-org/scancode-toolkit that referenced this pull request Mar 6, 2019
Based on @Freso PR at:
whipper-team/whipper#378

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@Freso Freso mentioned this pull request Aug 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Minor improvement to code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants