make it possible to and do build against a scalafix SNAPSHOT

- clients that do not override scalafixResolvers will continue working as sonatype snapshots is added by default - remove unnecessary scalafixResolvers overrides in scripted - failing tests were fixed by sticking to the plugin default
scalacenter · May 12, 2021 · a2cf9e2 · a2cf9e2
1 parent fdba46f
commit a2cf9e2
Show file tree

Hide file tree

Showing 9 changed files with 20 additions and 22 deletions.
diff --git a/project/Dependencies.scala b/project/Dependencies.scala
@@ -2,8 +2,8 @@ import sbt._
 
 object Dependencies {
   val x = List(1) // scalafix:ok
-  // when bumping remove dep on SNAPSHOT in sbt-1.5 scripted tests
-  def scalafixVersion: String = "0.9.27"
+  def scalafixVersion: String = "0.9.27+52-6c9eeec9-SNAPSHOT"
+
   val all = List(
     "org.eclipse.jgit" % "org.eclipse.jgit" % "5.11.0.202103091610-r",
     "ch.epfl.scala" % "scalafix-interfaces" % scalafixVersion,

diff --git a/src/main/scala/scalafix/sbt/ScalafixTestkitPlugin.scala b/src/main/scala/scalafix/sbt/ScalafixTestkitPlugin.scala
@@ -64,7 +64,8 @@ object ScalafixTestkitPlugin extends AutoPlugin {
             "scalafix-testkit.properties"
         IO.write(props, "Input data for scalafix testkit", out)
         List(out)
-      }
+      },
+      resolvers += Resolver.sonatypeRepo("snapshots")
     )
 
   private def semanticdbOption(

diff --git a/src/sbt-test/sbt-1.5/scala-3/project/plugins.sbt b/src/sbt-test/sbt-1.5/scala-3/project/plugins.sbt
@@ -1,6 +1,2 @@
 resolvers += Resolver.sonatypeRepo("public")
 addSbtPlugin("ch.epfl.scala" % "sbt-scalafix" % sys.props("plugin.version"))
-
-//FIXME: remove when scalafixVersion is bumped to 0.9.28
-resolvers += Resolver.sonatypeRepo("snapshots")
-dependencyOverrides += "ch.epfl.scala" % "scalafix-interfaces" % "0.9.27+52-6c9eeec9-SNAPSHOT"
diff --git a/src/sbt-test/sbt-1.5/scalafixEnable/project/plugins.sbt b/src/sbt-test/sbt-1.5/scalafixEnable/project/plugins.sbt
@@ -1,6 +1,2 @@
 resolvers += Resolver.sonatypeRepo("public")
 addSbtPlugin("ch.epfl.scala" % "sbt-scalafix" % sys.props("plugin.version"))
-
-//FIXME: remove when scalafixVersion is bumped to 0.9.28
-resolvers += Resolver.sonatypeRepo("snapshots")
-dependencyOverrides += "ch.epfl.scala" % "scalafix-interfaces" % "0.9.27+52-6c9eeec9-SNAPSHOT"
diff --git a/src/sbt-test/sbt-scalafix/basic/build.sbt b/src/sbt-test/sbt-scalafix/basic/build.sbt
@@ -12,8 +12,7 @@ inThisBuild(
     scalafixDependencies := List(
       // Custom rule published to Maven Central https://github.com/scalacenter/example-scalafix-rule
       "ch.epfl.scala" %% "example-scalafix-rule" % "1.4.0"
-    ),
-    scalafixResolvers := List(Repository.MavenCentral)
+    )
   )
 )
 

diff --git a/src/sbt-test/sbt-scalafix/concurrency/build.sbt b/src/sbt-test/sbt-scalafix/concurrency/build.sbt
@@ -2,6 +2,7 @@ val V = _root_.scalafix.sbt.BuildInfo
 
 scalaVersion := V.scala212
 libraryDependencies += "ch.epfl.scala" %% "scalafix-core" % V.scalafixVersion % ScalafixConfig
+resolvers += Resolver.sonatypeRepo("snapshots")
 
 // make it possible to run scalafix in parallel with other tasks via the `all` command
 // (work around "Cannot mix input tasks with plain tasks/settings")

diff --git a/src/sbt-test/sbt-scalafix/local-rules/build.sbt b/src/sbt-test/sbt-scalafix/local-rules/build.sbt
@@ -6,11 +6,6 @@ inThisBuild(
       // Custom rule cross-published to Maven Central https://github.com/scalacenter/example-scalafix-rule
       "ch.epfl.scala" %% "example-scalafix-rule" % "1.4.0"
     ),
-    // `Repository.central()` can only be used sbt 1.x / scala 2.12
-    // error: Static methods in interface require -target:jvm-1.8
-    scalafixResolvers := List(
-      coursierapi.MavenRepository.of("https://repo1.maven.org/maven2")
-    ),
     scalaVersion := "2.13.0", // out of sync with scalafix.sbt.BuildInfo.scala213 on purpose
     scalafixScalaBinaryVersion :=
       // this should be the default in sbt-scalafix 1.0

diff --git a/src/test/scala/scalafix/internal/sbt/SbtCompletionsSuite.scala b/src/test/scala/scalafix/internal/sbt/SbtCompletionsSuite.scala
@@ -1,6 +1,6 @@
 package scalafix.internal.sbt
 
-import coursierapi.Repository
+import coursierapi.{MavenRepository, Repository}
 import org.eclipse.jgit.lib.AbbreviatedObjectId
 import org.scalatest.Tag
 import sbt.complete.Parser
@@ -32,7 +32,12 @@ class SbtCompletionsSuite extends AnyFunSuite {
       .fromToolClasspath(
         "2.12",
         Seq(exampleDependency),
-        Seq(Repository.central)
+        Seq(
+          Repository.central,
+          MavenRepository.of(
+            "https://oss.sonatype.org/content/repositories/snapshots"
+          )
+        )
       )()
   val loadedRules = mainArgs.availableRules.toList
 

diff --git a/src/test/scala/scalafix/internal/sbt/ScalafixAPISuite.scala b/src/test/scala/scalafix/internal/sbt/ScalafixAPISuite.scala
@@ -3,7 +3,7 @@ package scalafix.internal.sbt
 import java.io.{ByteArrayOutputStream, PrintStream}
 import java.nio.file.Files
 
-import coursierapi.Repository
+import coursierapi.{MavenRepository, Repository}
 import org.scalactic.source.Position
 import sbt._
 import sbt.internal.sbtscalafix.Compat
@@ -31,7 +31,12 @@ class ScalafixAPISuite extends AnyFunSuite {
       .fromToolClasspath(
         "2.12",
         List("ch.epfl.scala" %% "example-scalafix-rule" % "1.4.0"),
-        Seq(Repository.central),
+        Seq(
+          Repository.central,
+          MavenRepository.of(
+            "https://oss.sonatype.org/content/repositories/snapshots"
+          )
+        ),
         logger
       )()
       .withArgs(Arg.PrintStream(new PrintStream(baos)))