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

Define test matrix coverage #118

Open
mattkram opened this issue Jul 8, 2023 · 4 comments
Open

Define test matrix coverage #118

mattkram opened this issue Jul 8, 2023 · 4 comments
Labels
automation CI/CD and automation tasks

Comments

@mattkram
Copy link
Contributor

mattkram commented Jul 8, 2023

We currently test across a range of conda and python versions. This means every time CI runs we run something like 80 parallel cases. We are also testing very old versions of conda.

I recommend we clearly defined what our supported versions are, and update our test matrix accordingly.

I recommend we test against:

  • Supported versions of conda
  • Supported versions of Python

And that we update this as those versions reach end of life and new versions become available.

@mattkram
Copy link
Contributor Author

mattkram commented Jul 8, 2023

I think I'm searching for a different term here. I'm thinking of "what is our promised level of support"?

@mattkram
Copy link
Contributor Author

mattkram commented Jul 8, 2023

Just looked and we are testing

  • 9 versions of conda
  • 3 operating systems
  • 3 versions of Python
os: ["ubuntu-latest", "macos-latest", "windows-latest"] 
python-version: [3.8, 3.9, "3.10"] 
conda-version: ["4.9", "4.10", "4.11", "4.12", "4.13", "4.14", "22.9", "22.11", "23.1"]

I recommend we keep the Python and OS as is. For conda I recommend we remove old versions and support the last 1-3 versions, up for discussion. I believe conda now releases quarterly.

@mattkram
Copy link
Contributor Author

mattkram commented Jul 8, 2023

Another option is to only test the oldest and newest versions of conda and Python supported before PRs are merged, and test the rest after merge into main. Theoretically we could have some errors and have to revert or change some merged behavior, but it may be rare enough to make it worthwhile?

@mattkram mattkram added the automation CI/CD and automation tasks label Jul 8, 2023
@AlbertDeFusco
Copy link
Contributor

AlbertDeFusco commented Jul 9, 2023

That sounds reasonable. I've not had to implement any version specific code so far anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation CI/CD and automation tasks
Projects
None yet
Development

No branches or pull requests

2 participants