aglinxinyuan commented on code in PR #4649:
URL: https://github.com/apache/texera/pull/4649#discussion_r3176337742
##########
build.sbt:
##########
@@ -19,13 +19,24 @@ ThisBuild / organization := "org.apache.texera"
ThisBuild / version := "1.1.0-incubating"
ThisBuild / scalaVersion := "2.13.18"
+// sbt-jacoco emits only HTML by default; add XML so Codecov can consume
+// per-module jacoco.xml at target/scala-2.13/jacoco/report/jacoco.xml.
+// JacocoPlugin defines a project-scoped default that overrides ThisBuild,
+// so this Seq is bundled into asfLicensingSettings (applied to every module).
+import com.github.sbt.jacoco.report.{JacocoReportFormats, JacocoReportSettings}
+lazy val coverageReportSettings = Seq(
+ jacocoReportSettings := JacocoReportSettings()
+ .withTitle("Apache Texera Coverage")
+ .withFormats(JacocoReportFormats.ScalaHTML, JacocoReportFormats.XML)
+)
+
// Per-module ASF licensing: each jar's META-INF/LICENSE describes only what
is in that jar.
// Modules without vendored code get Apache 2.0 only; workflow-operator
includes mbknor attribution.
// See project/AddMetaInfLicenseFiles.scala.
// Dist-producing modules additionally override Universal / mappings in their
own
// build.sbt (not here) — see AddMetaInfLicenseFiles.distMappings.
-lazy val asfLicensingSettings = AddMetaInfLicenseFiles.defaultSettings
-lazy val asfLicensingSettingsWithVendored =
AddMetaInfLicenseFiles.workflowOperatorSettings
+lazy val asfLicensingSettings = AddMetaInfLicenseFiles.defaultSettings ++
coverageReportSettings
+lazy val asfLicensingSettingsWithVendored =
AddMetaInfLicenseFiles.workflowOperatorSettings ++ coverageReportSettings
Review Comment:
Adding a comment to test if the merge is blocked.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
