Skip to content

Support Python 3.11 and improve dependency handling#3200

Merged
timothy-nunn merged 11 commits intoukaea:mainfrom
timothy-nunn:2950-support-python-311
Aug 9, 2024
Merged

Support Python 3.11 and improve dependency handling#3200
timothy-nunn merged 11 commits intoukaea:mainfrom
timothy-nunn:2950-support-python-311

Conversation

@timothy-nunn
Copy link
Copy Markdown
Collaborator

Description

Provides support for Python 3.11, making the minimum Python version 3.10.

Simplifies the testing and dependency installation while reviewing the dependency versions required for 3.10+.

@timothy-nunn timothy-nunn requested a review from jonmaddock May 29, 2024 10:03
@timothy-nunn timothy-nunn force-pushed the 2950-support-python-311 branch 2 times, most recently from f8f05c6 to 1ae4644 Compare May 31, 2024 14:00
@timothy-nunn timothy-nunn force-pushed the 2950-support-python-311 branch from 1ae4644 to 16502d8 Compare June 19, 2024 12:45
Copy link
Copy Markdown
Contributor

@jonmaddock jonmaddock left a comment

Choose a reason for hiding this comment

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

Looks great Tim! Could you make it clear in the docs (or comments) why we have dependencies spread across setup.py (and ["test", "examples"]), requirements.txt and requirements_dev.txt to help others understand the logic? Other than that, feel free to merge at will.

@timothy-nunn timothy-nunn force-pushed the 2950-support-python-311 branch 2 times, most recently from e5f0158 to 7c2c124 Compare August 9, 2024 09:58
@timothy-nunn timothy-nunn requested a review from jonmaddock August 9, 2024 10:38
@timothy-nunn timothy-nunn force-pushed the 2950-support-python-311 branch from 7c2c124 to 18fc6c9 Compare August 9, 2024 14:43
@timothy-nunn timothy-nunn dismissed jonmaddock’s stale review August 9, 2024 14:55

Implemented changes

@timothy-nunn timothy-nunn merged commit 3e873ff into ukaea:main Aug 9, 2024
@timothy-nunn timothy-nunn deleted the 2950-support-python-311 branch August 9, 2024 14:58
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.

3 participants