Test all supported versions of python #3724
Comments
|
Our support matrix is 3.8-3.12 right now. |
devinrsmith
added a commit
to devinrsmith/deephaven-core
that referenced
this issue
Mar 20, 2024
We've had a few recent issue/PRs dealing with python minimum version support: deephaven#5227, deephaven#5235, and deephaven#5271 Ultimately, we'd like a full matrix of testing support: deephaven#3724, deephaven#3725 In the more immediate term, there is a tool that I've verified is capable of catching these sorts of issues: https://github.com/netromdk/vermin. I've verified that it does catch deephaven#5227 and deephaven#5271, but does not catch deephaven#5235 (which seems like it is more of a runtime error). This PR adds a quick CI job to verify a python minimum version of 3.8. The `# novermin` comment is necessary in a few locations where we've explicitly / manually worked around python minimum version support.
devinrsmith
added a commit
that referenced
this issue
Mar 21, 2024
We've had a few recent issue/PRs dealing with python minimum version support: #5227, #5235, and #5271 Ultimately, we'd like a full matrix of testing support: #3724, #3725 In the more immediate term, there is a tool that I've verified is capable of catching these sorts of issues: https://github.com/netromdk/vermin. I've verified that it does catch #5227 and #5271, but does not catch #5235 (which seems like it is more of a runtime error). This PR adds a quick CI job to verify a python minimum version of 3.8. The `# novermin` comment is necessary in a few locations where we've explicitly / manually worked around python minimum version support.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, we only run unit and integration python tests with the version that comes in the base image(s) (3.10).
Ideally, we would be able to expand this to a matrix of all supported python versions.
Related #2593
The text was updated successfully, but these errors were encountered: