Skip to content

Commit

Permalink
internal (fix): Collect coverage report xml files properly (#3138)
Browse files Browse the repository at this point in the history
Due to the change in #3137, coverageAggregate now runs on Scala 3 and
chooses a different test report folder. This PR removes scala-2.xx or
scala-3 version name from the test report path.
  • Loading branch information
xerial committed Aug 12, 2023
1 parent 8cdd145 commit 5dc3ec1
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,9 @@ jobs:
- uses: codecov/codecov-action@v3
with:
name: airframe-coverage
file: ./projectJVM/target/scala-2.13/scoverage-report/scoverage.xml
file: ./projectJVM/target/scoverage-report/scoverage.xml
- name: Archive code coverage results
uses: actions/upload-artifact@v3
with:
name: code-coverage-report
path: ./projectJVM/target/scala-2.13/scoverage-report
path: ./projectJVM/target/scoverage-report
4 changes: 3 additions & 1 deletion build.sbt
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,9 @@ lazy val projectJVM =
.settings(noPublish)
.settings(
// Skip importing aggregated projects in IntelliJ IDEA
ideSkipProject := true
ideSkipProject := true,
// Use a stable coverage directory name without containing scala version
coverageDataDir := target.value
)
.aggregate(jvmProjects: _*)

Expand Down

0 comments on commit 5dc3ec1

Please sign in to comment.