Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Scala 2.13.4 #1290

Closed
wants to merge 1 commit into from

Conversation

daddykotex
Copy link
Contributor

Similar to #1179

@daddykotex
Copy link
Contributor Author

We use sbt-scalafix and the scalafixSemanticdb sbt key which seems to depend on this.

@daddykotex
Copy link
Contributor Author

I'm clearly missing a piece of the puzzle here.

sbt-scalafix defines the relevant keys like:

val scalafixSemanticdb: ModuleID =
      scalafixSemanticdb(BuildInfo.scalametaVersion)
def scalafixSemanticdb(scalametaVersion: String): ModuleID =
      "org.scalameta" % "semanticdb-scalac" % scalametaVersion cross CrossVersion.full

BuildInfo.scala defines scalametaVersion like def scalametaVersion: String = property("scalametaVersion") and in this case, property loads a properties files called: scalafix-interfaces.properties that seems to be coming from: https://github.com/scalacenter/scalafix/tree/master/scalafix-interfaces (back in this repo)

@daddykotex
Copy link
Contributor Author

if I add this diff:

diff --git a/build.sbt b/build.sbt
index 0a88d5f3..24f00d74 100644
--- a/build.sbt
+++ b/build.sbt
@@ -6,7 +6,7 @@ inThisBuild(
     crossScalaVersions := List(scala213, scala212, scala211),
     fork := true,
     semanticdbEnabled := true,
-    semanticdbVersion := scalafixSemanticdb.revision,
+    semanticdbVersion := "4.4.0",
     scalacOptions ++= List("-P:semanticdb:synthetics:on"),
     scalafixScalaBinaryVersion := scalaBinaryVersion.value,
     scalafixDependencies += "com.github.liancheng" %% "organize-imports" % "0.4.3"

it works fine, so I guess sbt-scalafix has to be updated beforehand.

@daddykotex
Copy link
Contributor Author

That would be a dependency cycle:

  1. the CI on this build needs sbt-scalafix scalafixSemanticdb to be updated
  2. scalafixSemanticdb depends on a property file generated in this build, in the interfaces module

@mlachkar
Copy link
Collaborator

You are right, there is a dependency cycle, butThe Scalafix plugin can be disabled until we finish this upgrade (or we fix manually the version of semanticdb for the plugin.)

@mlachkar
Copy link
Collaborator

I wanted to push on top of this commit, but by mistake I opened a new PR #1291

@daddykotex
Copy link
Contributor Author

No problem, thanks @mlachkar

@daddykotex daddykotex closed this Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants