-
Notifications
You must be signed in to change notification settings - Fork 5
Enable testing for py310 py311 and py313 in CI #202
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
base: main
Are you sure you want to change the base?
Conversation
55d1298 to
87c4ae4
Compare
.github/workflows/ci.yaml
Outdated
| strategy: | ||
| matrix: | ||
| python-version: ['3.10', '3.11', '3.12', '3.13'] | ||
| python-version: ['3.11'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be reverted
pyproject.toml
Outdated
| [tool.hatch.envs.default] | ||
| installer = "uv" | ||
| python = "3.12" | ||
| python = "python" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix this to something static. 3.10 ? 3.12 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we agreed to use py3.12
3e976bf to
624ca89
Compare
Due to the hatch setup the tests in the CI only ran on py312. It seems that hatch python discoverability detected a py312 version on the runner and therefore always defaulted to py312. It is needed to add to the python version to the test matrix to support the other python versions for running with hatch test.
624ca89 to
ff73dc0
Compare
ff73dc0 to
828038c
Compare
Our code fails for <py3.12. Somehow hatch runs on different python version. I don't know where it finds executable from. But see the CI from latest run on main for py3.11

I needed to add to the test matrix more versions and changed the default to use system python version. The fixes that need to be done in the code are marginal.