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
ci: expand build matrix for major OS #490
base: master
Are you sure you want to change the base?
Conversation
strategy: | ||
matrix: | ||
python-version: ["3.5", "3.7", "3.8", "3.9", "3.10", "3.11", "3.12"] | ||
fail-fast: false |
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.
fail-fast: false | |
fail-fast: true |
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.
I recommend false
, this way you get to see what exactly failed -- is it all versions lower than 3.10? Is it Windows?
With true
, if just the 3.5/Windows job failed, and all the others were cancelled, you don't get this useful info.
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.
true is by default, just make it explicit, I normally prefer to run all but it could yield quite extensive testing
@dbieber, mind having a look and sharing your thoughts? 🦩 |
I suggest including testing on Mac and Win.
also, the addition of OS builds does not need to be so extensive and could be listed in the
include
section adding 3.8 as the oldest and 3.12 as the latest python version