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

Shrink CI matrix #1547

Merged
merged 21 commits into from
Sep 11, 2024
Merged

Shrink CI matrix #1547

merged 21 commits into from
Sep 11, 2024

Conversation

lihaoyi
Copy link
Member

@lihaoyi lihaoyi commented Sep 11, 2024

  • We probably only need to each on each Java and Scala version once, rather than on the full cross matrix of all combinations.
  • We probably don't need to test every PR with every single Scala version. For now, for each Scala major version I picked three minor versions: highest, lowest, and one in the middle.
  • Replaced __.publishLocal with __.compile, which should hopefully catch most of the same bugs but be much cheaper not creating jars
  • I suspect the unitTest and integrationTest helper tasks were not doing the right thing - they were taking far longer than they should have been taking - but I couldn't figure out why. For now I just ditched them and use Mill queries to select the tests I want manually
  • Turned on Mill parallelism
  • We drop CI for Java 8, in line with the rest of the com-lihaoyi ecosystem

Hopefully that will give us enough confidence that things mostly work, and if anything emerges that's super version specific we can fix it ad-hoc without needing to wait 40+ hours of CPU time for every PR to be validated

@lihaoyi lihaoyi changed the title Shrink Mill CI matrix Shrink CI matrix Sep 11, 2024
@lihaoyi lihaoyi merged commit 2668a43 into main Sep 11, 2024
16 checks passed
@lihaoyi lihaoyi deleted the shrink-ci branch September 11, 2024 23:36
@lefou lefou added this to the 3.0.0-M3 milestone Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants