Skip to content

Commit 26da8f8

Browse files
author
Tim Sheppard
committed
Check for empty value of exclusion options and only enable if non empty
1 parent 1e015f8 commit 26da8f8

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

build.sbt

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name := "sbt-scoverage"
22

33
organization := "org.scoverage"
44

5-
version := "0.99.10"
5+
version := "0.99.11-SNAPSHOT"
66

77
scalaVersion := "2.10.4"
88

src/main/scala/scoverage/ScoverageSbtPlugin.scala

+9-3
Original file line numberDiff line numberDiff line change
@@ -85,8 +85,14 @@ class ScoverageSbtPlugin extends sbt.Plugin {
8585
s"-Xplugin:${classpath.getAbsolutePath}",
8686
s"-P:scoverage:dataDir:${crossTarget.value.getAbsolutePath}/scoverage-data"
8787
) ++
88-
Option(excludedPackages.value).map(v => s"-P:scoverage:excludedPackages:$v") ++
89-
Option(scoverageExcludedFiles.value).map(v => s"-P:scoverage:excludedFiles:$v")
88+
Option(excludedPackages.value.trim).map{
89+
case "" => ""
90+
case v => s"-P:scoverage:excludedPackages:$v"
91+
} ++
92+
Option(scoverageExcludedFiles.value.trim).map{
93+
case "" => ""
94+
case v => s"-P:scoverage:excludedFiles:$v"
95+
}
9096
}
9197
},
9298

@@ -436,4 +442,4 @@ class ScoverageSbtPlugin extends sbt.Plugin {
436442
// // test in ScoverageCompile <<= (test in ScoverageTest)
437443
// // )
438444
//
439-
//}
445+
//}

0 commit comments

Comments
 (0)