Skip to content

Cleanup old support for Python 3.6 and earlier code - #8318

Closed
jamesobutler wants to merge 1 commit into
numba:mainfrom
jamesobutler:drop-pre-python-3.7
Closed

jamesobutler wants to merge 1 commit into
numba:mainfrom
jamesobutler:drop-pre-python-3.7

Conversation

@jamesobutler

Copy link
Copy Markdown
Contributor

Numba set the minimum required Python version to 3.7 in 7bdd800. This PR removes extra code that won't be executed based on this current minimum requirement of Python 3.7.

numba/numba/__init__.py

Lines 140 to 141 in aaa6a00

if PYVERSION < (3, 7):
raise ImportError("Numba needs Python 3.7 or greater")

Numba set the minimum required Python version to 3.7 in numba@7bdd800.
@jamesobutler

Copy link
Copy Markdown
Contributor Author

Similar to changes made in #7920, but that PR has seemingly become stale? I did not see this existing PR because it had fallen down to the 3rd page of the open PRs. Feel free to close this as a duplicate.

@esc esc self-assigned this Aug 8, 2022
@esc
esc removed request for sklam and stuartarchibald August 8, 2022 16:53
@esc

esc commented Aug 8, 2022

Copy link
Copy Markdown
Member

@jamesobutler thank you for submitting this. I plan to look at this PR soon.

@esc

esc commented Aug 15, 2022

Copy link
Copy Markdown
Member

@jamesobutler thank you for submitting this. I compared this with #7920 today. I think we will need to adopt some of the diff of this PR.

@esc esc added 4 - Waiting on reviewer Waiting for reviewer to respond to author and removed 2 - In Progress labels Aug 15, 2022
@esc

esc commented Aug 18, 2022

Copy link
Copy Markdown
Member

@jamesobutler #7920 has been updated to include the "missing" changes from this PR. Would you have a spare moment to cross-check it? That would be most appreciated here since you've looked into this already! Thank you! 🙏

@esc

esc commented Aug 19, 2022

Copy link
Copy Markdown
Member

@jamesobutler thank you again for your review of #7920! I will close this now as duplicate.

@esc esc closed this Aug 19, 2022
@esc esc added duplicate abandoned PR is abandoned (no reason required) and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author labels Aug 19, 2022
@jamesobutler
jamesobutler deleted the drop-pre-python-3.7 branch August 19, 2022 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

abandoned PR is abandoned (no reason required) duplicate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants