Split CI into multiple steps

.github/workflows/ci.yml

@@ -29,7 +29,7 @@ jobs:
         os: [ubuntu-latest]
         scala: [2.12.15]
         java: [temurin@8]
-        ci: [ciJVM]
+        project: [rootJVM]
     runs-on: ${{ matrix.os }}
     steps:
       - name: Checkout current branch (full)
@@ -57,9 +57,15 @@ jobs:
           key: ${{ runner.os }}-sbt-cache-v2-${{ hashFiles('**/*.sbt') }}-${{ hashFiles('project/build.properties') }}
 
       - name: Check that workflows are up to date
-        run: sbt '++${{ matrix.scala }}' 'project /' githubWorkflowCheck
+        run: sbt 'project ${{ matrix.project }}' '++${{ matrix.scala }}' 'project /' githubWorkflowCheck
 
-      - run: sbt '++${{ matrix.scala }}' '${{ matrix.ci }}'
+      - run: sbt 'project ${{ matrix.project }}' '++${{ matrix.scala }}' headerCheckAll scalafmtCheckAll scalafmtSbtCheck
+
+      - run: sbt 'project ${{ matrix.project }}' '++${{ matrix.scala }}' clean
+
+      - run: sbt 'project ${{ matrix.project }}' '++${{ matrix.scala }}' test
+
+      - run: sbt 'project ${{ matrix.project }}' '++${{ matrix.scala }}' mimaReportBinaryIssues
 
       - name: Make target directories
         run: mkdir -p github/target github-actions/target kernel/target versioning/target ci-release/target target .js/target ci-signing/target mima/target .jvm/target .native/target no-publish/target sonatype/target ci/target sonatype-ci-release/target core/target settings/target project/target

ci/src/main/scala/org/typelevel/sbt/CrossRootProject.scala

@@ -18,8 +18,6 @@ package org.typelevel.sbt
 
 import sbt._
 import org.typelevel.sbt.gha.GenerativePlugin.autoImport._
-import TypelevelCiPlugin.ciCommands
-import TypelevelKernelPlugin.mkCommand
 
 /**
  * Simultaneously creates a `root`, `rootJVM`, `rootJS`, and `rootNative` project, and
@@ -113,12 +111,8 @@ object TypelevelCiCrossPlugin extends AutoPlugin {
   override def requires = TypelevelCiPlugin
 
   override def buildSettings = Seq(
-    githubWorkflowBuild ~= { steps =>
-      // remove the usual ci step and replace with matrix ci
-      steps.diff(Seq(WorkflowStep.Sbt(List("ci")))) :+
-        WorkflowStep.Sbt(List(s"$${{ matrix.ci }}"))
-    },
-    githubWorkflowBuildMatrixAdditions += "ci" -> Nil
+    githubWorkflowBuildSbtStepPreamble ~= { s"project $${{ matrix.project }}" +: _ },
+    githubWorkflowBuildMatrixAdditions += "project" -> Nil
   )
 }
 
@@ -127,56 +121,56 @@ object TypelevelCiCrossPlugin extends AutoPlugin {
 object TypelevelCiJVMPlugin extends AutoPlugin {
   override def requires = TypelevelCiCrossPlugin
 
-  override def buildSettings: Seq[Setting[_]] =
-    addCommandAlias("ciJVM", mkCommand(ciJVMCommands)) ++ Seq(
-      githubWorkflowBuildMatrixAdditions ~= { matrix =>
-        matrix.updated("ci", matrix("ci") ::: "ciJVM" :: Nil)
-      }
-    )
-
-  val ciJVMCommands = "project rootJVM" :: ciCommands.tail
+  override def buildSettings: Seq[Setting[_]] = Seq(
+    githubWorkflowBuildMatrixAdditions ~= { matrix =>
+      matrix.updated("project", matrix("project") ::: "rootJVM" :: Nil)
+    }
+  )
 }
 
 object TypelevelCiJSPlugin extends AutoPlugin {
   override def requires = TypelevelCiCrossPlugin
 
-  override def buildSettings: Seq[Setting[_]] =
-    addCommandAlias("ciJS", mkCommand(ciJSCommands)) ++ Seq(
-      githubWorkflowBuildMatrixAdditions ~= { matrix =>
-        matrix.updated("ci", matrix("ci") ::: "ciJS" :: Nil)
-      },
-      githubWorkflowBuildMatrixExclusions ++= {
-        githubWorkflowJavaVersions
-          .value
-          .tail
-          .map(java => MatrixExclude(Map("ci" -> "ciJS", "java" -> java.render)))
+  override def buildSettings: Seq[Setting[_]] = Seq(
+    githubWorkflowBuildMatrixAdditions ~= { matrix =>
+      matrix.updated("project", matrix("project") ::: "rootJS" :: Nil)
+    },
+    githubWorkflowBuildMatrixExclusions ++= {
+      githubWorkflowJavaVersions
+        .value
+        .tail
+        .map(java => MatrixExclude(Map("project" -> "rootJS", "java" -> java.render)))
+    },
+    githubWorkflowBuild ~= { steps =>
+      steps.flatMap {
+        case testStep @ WorkflowStep.Sbt(List("test"), _, _, _, _, _) =>
+          List(WorkflowStep.Sbt(List("Test/fastOptJS")), testStep)
+        case step => List(step)
       }
-    )
+    }
+  )
 
-  val ciJSCommands = "project rootJS" :: ciCommands.tail.flatMap {
-    case "test" => List("Test/fastOptJS", "test")
-    case x => List(x)
-  }
 }
 
 object TypelevelCiNativePlugin extends AutoPlugin {
   override def requires = TypelevelCiCrossPlugin
 
-  override def buildSettings: Seq[Setting[_]] =
-    addCommandAlias("ciNative", mkCommand(ciNativeCommands)) ++ Seq(
-      githubWorkflowBuildMatrixAdditions ~= { matrix =>
-        matrix.updated("ci", matrix("ci") ::: "ciNative" :: Nil)
-      },
-      githubWorkflowBuildMatrixExclusions ++= {
-        githubWorkflowJavaVersions
-          .value
-          .tail
-          .map(java => MatrixExclude(Map("ci" -> "ciNative", "java" -> java.render)))
+  override def buildSettings: Seq[Setting[_]] = Seq(
+    githubWorkflowBuildMatrixAdditions ~= { matrix =>
+      matrix.updated("project", matrix("project") ::: "rootNative" :: Nil)
+    },
+    githubWorkflowBuildMatrixExclusions ++= {
+      githubWorkflowJavaVersions
+        .value
+        .tail
+        .map(java => MatrixExclude(Map("project" -> "rootNative", "java" -> java.render)))
+    },
+    githubWorkflowBuild ~= { steps =>
+      steps.flatMap {
+        case testStep @ WorkflowStep.Sbt(List("test"), _, _, _, _, _) =>
+          List(WorkflowStep.Sbt(List("Test/nativeLink")), testStep)
+        case step => List(step)
       }
-    )
-
-  val ciNativeCommands = "project rootNative" :: ciCommands.tail.flatMap {
-    case "test" => List("Test/nativeLink", "test")
-    case x => List(x)
-  }
+    }
+  )
 }

ci/src/main/scala/org/typelevel/sbt/TypelevelCiPlugin.scala

@@ -21,7 +21,6 @@ import org.typelevel.sbt.gha.GenerativePlugin
 import org.typelevel.sbt.gha.GitHubActionsPlugin
 import org.typelevel.sbt.gha.GenerativePlugin.autoImport._
 import com.typesafe.tools.mima.plugin.MimaPlugin
-import TypelevelKernelPlugin.mkCommand
 
 object TypelevelCiPlugin extends AutoPlugin {
 
@@ -32,32 +31,28 @@ object TypelevelCiPlugin extends AutoPlugin {
     def tlCrossRootProject: CrossRootProject = CrossRootProject()
   }
 
-  override def buildSettings =
-    addCommandAlias("ci", mkCommand(ciCommands)) ++ Seq(
-      githubWorkflowPublishTargetBranches := Seq(),
-      githubWorkflowBuild := Seq(WorkflowStep.Sbt(List("ci"))),
-      githubWorkflowJavaVersions := Seq(JavaSpec.temurin("8")),
-      githubWorkflowGeneratedUploadSteps ~= { steps =>
-        // hack hack hack until upstreamed
-        // workaround for https://github.com/djspiewak/sbt-github-actions/pull/66
-        steps.flatMap {
-          case compressStep @ WorkflowStep
-                .Run(command :: _, _, Some("Compress target directories"), _, _, _) =>
-            val mkdirStep = WorkflowStep.Run(
-              commands = List(command.replace("tar cf targets.tar", "mkdir -p")),
-              name = Some("Make target directories")
-            )
-            List(mkdirStep, compressStep)
-          case step => List(step)
-        }
+  override def buildSettings = Seq(
+    githubWorkflowPublishTargetBranches := Seq(),
+    githubWorkflowBuild := Seq(
+      WorkflowStep.Sbt(List("clean")),
+      WorkflowStep.Sbt(List("test")),
+      WorkflowStep.Sbt(List("mimaReportBinaryIssues"))
+    ),
+    githubWorkflowJavaVersions := Seq(JavaSpec.temurin("8")),
+    githubWorkflowGeneratedUploadSteps ~= { steps =>
+      // hack hack hack until upstreamed
+      // workaround for https://github.com/djspiewak/sbt-github-actions/pull/66
+      steps.flatMap {
+        case compressStep @ WorkflowStep
+              .Run(command :: _, _, Some("Compress target directories"), _, _, _) =>
+          val mkdirStep = WorkflowStep.Run(
+            commands = List(command.replace("tar cf targets.tar", "mkdir -p")),
+            name = Some("Make target directories")
+          )
+          List(mkdirStep, compressStep)
+        case step => List(step)
       }
-    )
-
-  val ciCommands = List(
-    "project /",
-    "clean",
-    "test",
-    "mimaReportBinaryIssues"
+    }
   )
 
 }

core/src/main/scala/org/typelevel/sbt/TypelevelPlugin.scala

@@ -20,8 +20,6 @@ import sbt._, Keys._
 import org.typelevel.sbt.gha.GenerativePlugin
 import org.typelevel.sbt.gha.GitHubActionsPlugin
 import de.heikoseeberger.sbtheader.AutomateHeaderPlugin
-import TypelevelCiPlugin.ciCommands
-import TypelevelKernelPlugin.mkCommand
 
 object TypelevelPlugin extends AutoPlugin {
 
@@ -39,7 +37,6 @@ object TypelevelPlugin extends AutoPlugin {
   }
 
   import autoImport._
-  import TypelevelKernelPlugin.autoImport._
   import TypelevelSettingsPlugin.autoImport._
   import TypelevelSonatypeCiReleasePlugin.autoImport._
   import GenerativePlugin.autoImport._
@@ -62,38 +59,12 @@ object TypelevelPlugin extends AutoPlugin {
         scala <- githubWorkflowScalaVersions.value.init
         java <- githubWorkflowJavaVersions.value.tail
       } yield MatrixExclude(Map("scala" -> scala, "java" -> java.render))
+    },
+    githubWorkflowBuild ~= { steps =>
+      WorkflowStep.Sbt(List("headerCheckAll", "scalafmtCheckAll", "scalafmtSbtCheck")) +: steps
     }
-  ) ++ tlReplaceCommandAlias("ci", mkCommand(fmtCheckCommands ::: ciCommands.tail))
+  )
 
   override def projectSettings = AutomateHeaderPlugin.projectSettings
 
-  val fmtCheckCommands =
-    List("project /", "headerCheckAll", "scalafmtCheckAll", "scalafmtSbtCheck")
-}
-
-import TypelevelKernelPlugin.autoImport._
-import TypelevelPlugin.fmtCheckCommands
-import TypelevelCiJVMPlugin.ciJVMCommands
-import TypelevelCiJSPlugin.ciJSCommands
-import TypelevelCiNativePlugin.ciNativeCommands
-
-object TypelevelJVMPlugin extends AutoPlugin {
-  override def requires = TypelevelCiJVMPlugin
-  override def trigger = allRequirements
-  override def buildSettings =
-    tlReplaceCommandAlias("ciJVM", mkCommand(fmtCheckCommands ++ ciJVMCommands))
-}
-
-object TypelevelJSPlugin extends AutoPlugin {
-  override def requires = TypelevelCiJSPlugin
-  override def trigger = allRequirements
-  override def buildSettings =
-    tlReplaceCommandAlias("ciJS", mkCommand(fmtCheckCommands ++ ciJSCommands))
-}
-
-object TypelevelNativePlugin extends AutoPlugin {
-  override def requires = TypelevelCiNativePlugin
-  override def trigger = allRequirements
-  override def buildSettings =
-    tlReplaceCommandAlias("ciNative", mkCommand(fmtCheckCommands ++ ciNativeCommands))
 }