Skip to content

Commit 6f74aed

Browse files
cacocojenkins
authored andcommitted
Revert to scala 2.12.12 due to scoverage issue with scala 2.12.13
Problem/Solution The scoverage plugin has an issue due to a binary incompatibility introduced in scala 2.12.13 (see: scoverage/sbt-scoverage#319). It has been patched in the plugin in scoverage/scalac-scoverage-plugin#306 which is currently awaiting a release. However after updating our projects to scala 2.12.13, coverage is broken and failing the CI builds. Downgrade to scala 2.12.12 until the scoverage plugin has been published with the fix. Differential Revision: https://phabricator.twitter.biz/D635917
1 parent 788e579 commit 6f74aed

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

.github/workflows/continuous-integration.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ jobs:
1919
strategy:
2020
fail-fast: false
2121
matrix:
22-
scala: [2.12.13, 2.13.1]
22+
scala: [2.12.12, 2.13.1]
2323
java: ['1.8', '1.11']
2424
runs-on: ubuntu-latest
2525
steps:

CHANGELOG.rst

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ Breaking API Changes
2222
Changed
2323
~~~~~~~
2424

25+
* finatra: Revert to scala version 2.12.12 due to https://github.com/scoverage/sbt-scoverage/issues/319
26+
``PHAB_ID=D635917``
27+
2528
* finatra: Bump scala version to 2.12.13 ``PHAB_ID=D632567``
2629

2730
* finatra: Move com.twitter.finatra.http.{jsonpatch,request} from the finatra/http-server project to

build.sbt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@ val releaseVersion = "21.3.0-SNAPSHOT"
88

99
lazy val buildSettings = Seq(
1010
version := releaseVersion,
11-
scalaVersion := "2.12.13",
12-
crossScalaVersions := Seq("2.12.13", "2.13.1"),
11+
scalaVersion := "2.12.12",
12+
crossScalaVersions := Seq("2.12.12", "2.13.1"),
1313
scalaModuleInfo := scalaModuleInfo.value.map(_.withOverrideScalaVersion(true)),
1414
fork in Test := true, // We have to fork to get the JavaOptions
1515
javaOptions in Test ++= travisTestJavaOptions,

0 commit comments

Comments
 (0)