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

Adjust supported python versions to ^3.8 #312

Merged
merged 9 commits into from Feb 22, 2024

Conversation

jhamon
Copy link
Collaborator

@jhamon jhamon commented Feb 14, 2024

Problem

Extend test coverage for python 3.11 and 3.12. Adjust pyproject.toml python dependency to ^3.8 to ease some installation problems in projects which depend on us.

Solution

Tweak test matrix to add coverage for 3.11 and 3.12. Adjust pyproject.toml to reflect specific version deps for grpcio and urllib3.

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Infrastructure change (CI configs, etc)

Test Plan

Describe specific steps for validating this change.

@jhamon jhamon force-pushed the jhamon/explore-python-dep-versions branch from 2c57361 to d317cde Compare February 21, 2024 18:14
@jhamon jhamon force-pushed the jhamon/explore-python-dep-versions branch from 69b9dfc to 5a6c5c6 Compare February 21, 2024 21:59
Comment on lines +106 to +107
- 1.59.3
- 1.60.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This range is different for 3.11 and 3.12 vs earlier versions.

@@ -47,15 +48,21 @@ classifiers=[
]

[tool.poetry.dependencies]
python = ">=3.8,<3.13"
python = "^3.8"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This was the requested change that motivated me to start poking around with this.

pytest-asyncio = "0.15.1"
pytest-cov = "2.10.1"
pytest-mock = "3.6.1"
pytest-timeout = ">=1.4.2"
pytest-timeout = "2.2.0"
Copy link
Collaborator Author

@jhamon jhamon Feb 22, 2024

Choose a reason for hiding this comment

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

We needed to bump these pytest dev dependencies to work with python 3.12.

@jhamon jhamon changed the title Adjust supported python versions Adjust supported python versions to ^3.8 Feb 22, 2024
@jhamon jhamon marked this pull request as ready for review February 22, 2024 00:16
Copy link
Contributor

@austin-denoble austin-denoble left a comment

Choose a reason for hiding this comment

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

Nice work, I like the cleanup in the GH workflows and moving the dependency tests into specific actions. 🚢

@jhamon jhamon merged commit 0d3cc45 into main Feb 22, 2024
124 checks passed
@jhamon jhamon deleted the jhamon/explore-python-dep-versions branch February 22, 2024 15:30
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.

None yet

2 participants