From c57e764c40d85ea51988650337866b607ac924fa Mon Sep 17 00:00:00 2001 From: Milad Khajavi Date: Tue, 11 Apr 2023 12:57:47 +0330 Subject: [PATCH] add default strategy for testing modules when ciTargetScalaVersions is empty. --- .github/workflows/ci.yml | 7 +---- build.sbt | 9 +----- zio-sbt-ci/src/main/scala/zio/sbt/V.scala | 2 +- .../main/scala/zio/sbt/ZioSbtCiPlugin.scala | 29 ++++++++++++++++++- 4 files changed, 31 insertions(+), 16 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 66255112..cc66aa60 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -73,11 +73,6 @@ jobs: - '8' - '11' - '17' - scala-project: - - ++2.12.17 zio-sbt-website - - ++2.12.17 zio-sbt-ecosystem - - ++2.12.17 zio-sbt-ci - - ++2.12.17 zio-sbt-tests steps: - name: Install libuv run: sudo apt-get update && sudo apt-get install -y libuv1-dev @@ -94,7 +89,7 @@ jobs: with: fetch-depth: '0' - name: Test - run: sbt ${{ matrix.scala-project }}/test + run: sbt ++test update-readme: name: Update README runs-on: ubuntu-latest diff --git a/build.sbt b/build.sbt index 20d9d95f..47e030f2 100644 --- a/build.sbt +++ b/build.sbt @@ -16,14 +16,7 @@ inThisBuild( developers := List( Developer("khajavi", "Milad Khajavi", "khajavi@gmail.com", url("https://github.com/khajavi")) ), - ciEnabledBranches := Seq("main"), -// ciTargetScalaVersions := -// targetScalaVersionsFor( -// `zio-sbt-website`, -// `zio-sbt-ecosystem`, -// `zio-sbt-ci`, -// `zio-sbt-tests` -// ).value, + ciEnabledBranches := Seq("main"), ciCheckGithubWorkflowSteps := Seq.empty ) ) diff --git a/zio-sbt-ci/src/main/scala/zio/sbt/V.scala b/zio-sbt-ci/src/main/scala/zio/sbt/V.scala index fbdbfbb0..6e14ba5a 100644 --- a/zio-sbt-ci/src/main/scala/zio/sbt/V.scala +++ b/zio-sbt-ci/src/main/scala/zio/sbt/V.scala @@ -10,5 +10,5 @@ object V { "actions/setup-java" -> "v3.10.0", "coursier/cache-action" -> "v6", "actions/setup-node" -> "v3" - )(packageName) + ).map { case (k, v) => (k, s"$k@$v") }.apply(packageName) } diff --git a/zio-sbt-ci/src/main/scala/zio/sbt/ZioSbtCiPlugin.scala b/zio-sbt-ci/src/main/scala/zio/sbt/ZioSbtCiPlugin.scala index d6c8d5ee..2f337de5 100644 --- a/zio-sbt-ci/src/main/scala/zio/sbt/ZioSbtCiPlugin.scala +++ b/zio-sbt-ci/src/main/scala/zio/sbt/ZioSbtCiPlugin.scala @@ -291,7 +291,34 @@ object ZioSbtCiPlugin extends AutoPlugin { ) ++ extraTestSteps ) - Seq(if (groupSimilarTests) GroupTests else FlattenTests) + val DefaultTestStrategy = + Job( + id = "test", + name = "Test", + strategy = Some( + Strategy( + matrix = Map("java" -> javaPlatforms.toList), + maxParallel = matrixMaxParallel, + failFast = false + ) + ), + steps = (if (swapSizeGB > 0) Seq(setSwapSpace) else Seq.empty) ++ + Seq( + SetupLibuv, + SetupJava("${{ matrix.java }}"), + CacheDependencies, + checkout, + Step.SingleStep( + name = "Test", + run = Some(prefixJobs + "sbt ++test") + ) + ) ++ extraTestSteps + ) + + if (javaPlatformMatrix.isEmpty && scalaVersionMatrix.isEmpty) + Seq(DefaultTestStrategy) + else + Seq(if (groupSimilarTests) GroupTests else FlattenTests) } lazy val reportSuccessfulJobs: Def.Initialize[Seq[Job]] = Def.setting {