ci: simplify cross platform testing, option 2#2723
Closed
ChristianMurphy wants to merge 1 commit into
Closed
Conversation
2c8c852 to
3005fe2
Compare
2 tasks
3005fe2 to
9f50eb4
Compare
This uses a T shaped build matrix to cover a large number of configurations. :notebook: this adds several new Java distributions we didn't previously test On Linux and Java 11, test all Java distributions. On Linux and adopt open jdk, test Java 8. On adopt open jdk and java 11, test MacOS and Windows. Before there were 18 jobs. With this change there are 8 jobs.
9f50eb4 to
3b1ab88
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Checklist
Description of change
Alternative to #2722
This uses a T shaped build matrix to cover a large number of configurations.
📓 this adds several new Java distributions we didn't previously test
Before there were 18 jobs.
With this change there are 8 jobs.