Expanded the matrix to include macos and windows tests
Removed unnecessary lines from the matrix definition
How I did it
All default values must be in the top-level matrix definition
The includes section should contain alternative runs - these will not have all default values defined
Therefore reading from the matrix requires passing default values again
However we can still delete a lot of repetition
How to verify it
Tests should pass. We should have 2 extra jobs and the existing ones should keep on existing.
Commit message
- Expanded the matrix to include macos and windows tests
- Refactor the matrix and removed unnecessary lines
- Remove now-redundant windows job
Note that all because of the way github creates matrix jobs, default
values must be in the top-level matrix definition. The includes section
should contain alternative runs - these will not have all default values
defined, therefore reading from the matrix requires passing default
values again.
What I did
How I did it
includes
section should contain alternative runs - these will not have all default values definedHow to verify it
Commit message
Description for the changelog
N/A
Cute Animal Picture