Skip to content

Commit

Permalink
bugfix: Don't use aggregate in Scala Native
Browse files Browse the repository at this point in the history
This causes the nativeLink task to be executed for all the modules and it doesn't make sense to do that for non app modules.
  • Loading branch information
tgodzik authored and kitbellew committed Feb 17, 2025
1 parent 7a3419a commit b822e23
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions build.sbt
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ lazy val cli = crossProject(JVMPlatform, NativePlatform, JSPlatform)
// TODO: enable NPM publishing
.jsSettings(scalaJsSettings, scalaJSUseMainModuleInitializer := true)
.jvmEnablePlugins(NativeImagePlugin)
.jvmConfigure(_.dependsOn(dynamic.jvm).aggregate(dynamic.jvm)).aggregate(core)
.jvmConfigure(_.dependsOn(dynamic.jvm).aggregate(dynamic.jvm))

lazy val tests = crossProject(JVMPlatform, NativePlatform, JSPlatform)
.withoutSuffixFor(JVMPlatform).in(file("scalafmt-tests")).settings(
Expand All @@ -246,7 +246,7 @@ lazy val tests = crossProject(JVMPlatform, NativePlatform, JSPlatform)
}),
).enablePlugins(BuildInfoPlugin)
.jvmSettings(javaOptions += "-Dfile.encoding=UTF8").dependsOn(core)
.jsSettings(scalaJsSettings).jsEnablePlugins(ScalaJSPlugin)
.aggregate(core).jsSettings(scalaJsSettings).jsEnablePlugins(ScalaJSPlugin)

lazy val sharedTestSettings = Seq(libraryDependencies += munit.value % Test)

Expand Down

0 comments on commit b822e23

Please sign in to comment.