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

Trove classifiers may be out of date. #580

Merged
merged 2 commits into from
Apr 30, 2020

Conversation

esc
Copy link
Member

@esc esc commented Apr 21, 2020

https://github.com/numba/llvmlite/blob/master/setup.py#L181-L183

It should probably read: 3.6 3.7 3.8. I can prep a PR if deemed correct.

@stuartarchibald
Copy link
Contributor

Yes, and yes please, thanks.

@stuartarchibald
Copy link
Contributor

Can get rid of https://github.com/numba/llvmlite/blob/master/setup.py#L165-L166 whilst at it ?

@esc
Copy link
Member Author

esc commented Apr 21, 2020

@stuartarchibald yeah, I saw that and considered it and will now include it.

esc added 2 commits April 21, 2020 10:21
This is set to match the Python versions supported by Numba.
Since 3.4 is no longer a supported Python version, we can safely remove
dependencies needed on that version.
@esc
Copy link
Member Author

esc commented Apr 21, 2020

I have converted the issue to a pull-request and it is now read for review.

@esc
Copy link
Member Author

esc commented Apr 21, 2020

I should also note that, the setup.py still contains some flake8 issues that need to be addressed in a subsequent PR.

@jccurtis
Copy link

Excellent, thanks folks! I was just about to make this PR. This causes the release on https://pypi.org/project/llvmlite/ to let python2.7 think it can use it. I know this is deprecated but it has caused some confusing CI failures since LLVM is not installed in the CI chain. Thanks @esc and @stuartarchibald. Is there a plan to bump the version on pypi?

Copy link
Contributor

@stuartarchibald stuartarchibald left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks.

@stuartarchibald stuartarchibald added this to the Version v0.32.1 milestone Apr 30, 2020
@sklam sklam merged commit 43439ca into numba:master Apr 30, 2020
sklam added a commit that referenced this pull request Apr 30, 2020
Trove classifiers may be out of date.
@esc esc deleted the fix/python_versions_setup.py branch June 16, 2020 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants