bump scalafix to 0.6.0-M8

Author: MasseGuillaume

scalafix/build.sbt

@@ -1,23 +1,31 @@
 def scalafixVersion = _root_.scalafix.Versions.version
-inScope(Global)(
-  List(
-    scalaVersion := _root_.scalafix.Versions.scala212
-  )
+
+lazy val baseSettings = Seq(
+  scalaVersion := _root_.scalafix.Versions.scala212
 )
 
 lazy val root = project
   .in(file("."))
-  .aggregate(
-    rules, input, output, tests
-  )
+  .settings(baseSettings)
+  .aggregate(rules, input, output, tests)
 
-lazy val rules = project.settings(
-  libraryDependencies += "ch.epfl.scala" %% "scalafix-core" % scalafixVersion
-)
+lazy val rules = project
+  .settings(baseSettings)
+  .settings(
+    libraryDependencies += "ch.epfl.scala" %% "scalafix-core" % scalafixVersion
+  )
 
 lazy val input = project
+.settings(baseSettings)
   .settings(
-    scalafixSourceroot := sourceDirectory.in(Compile).value
+    addCompilerPlugin(scalafixSemanticdb),
+    scalacOptions ++= {
+      val sourceroot = sourceDirectory.in(Compile).value / "scala"
+      Seq(
+        "-Yrangepos",
+        s"-P:semanticdb:sourceroot:$sourceroot"
+      )
+    }
   )
 
 lazy val output = project
@@ -27,18 +35,15 @@ lazy val output = project
   )
 
 lazy val tests = project
+  .settings(baseSettings)
   .settings(
     libraryDependencies += "ch.epfl.scala" % "scalafix-testkit" % scalafixVersion % Test cross CrossVersion.full,
-    buildInfoPackage := "fix",
-    buildInfoKeys := Seq[BuildInfoKey](
-      "inputSourceroot" ->
-        sourceDirectory.in(input, Compile).value,
-      "outputSourceroot" ->
-        sourceDirectory.in(output, Compile).value,
-      "inputClassdirectory" ->
-        classDirectory.in(input, Compile).value
-    ),
-    test in Test := (test in Test).dependsOn(compile in (output, Compile)).value
+    scalafixTestkitOutputSourceDirectories :=
+      sourceDirectories.in(output, Compile).value,
+    scalafixTestkitInputSourceDirectories :=
+      sourceDirectories.in(input, Compile).value,
+    scalafixTestkitInputClasspath :=
+      fullClasspath.in(input, Compile).value
   )
   .dependsOn(input, rules)
-  .enablePlugins(BuildInfoPlugin)
+  .enablePlugins(ScalafixTestkitPlugin)

scalafix/project/plugins.sbt

@@ -1,3 +1,3 @@
-addSbtPlugin("ch.epfl.scala" % "sbt-scalafix" % "0.5.10")
+addSbtPlugin("ch.epfl.scala" % "sbt-scalafix" % "0.6.0-M8")
 addSbtPlugin("io.get-coursier" % "sbt-coursier" % "1.0.0")
 addSbtPlugin("com.eed3si9n" % "sbt-buildinfo" % "0.6.1")

scalafix/rules/src/main/scala/fix/Scalacollectioncompat_newcollections.scala

@@ -49,13 +49,13 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
       Symbol("_root_.scala.collection.mutable.SetLike.retain.")
     )
 
-  def replaceMutableSet(ctx: RuleCtx) =
+  def replaceMutableSet(ctx: RuleCtx): Patch = 
     ctx.tree.collect {
       case retainSet(n: Name) =>
         ctx.replaceTree(n, "filterInPlace")
     }.asPatch
 
-  def replaceMutableMap(ctx: RuleCtx) =
+  def replaceMutableMap(ctx: RuleCtx): Patch = 
     ctx.tree.collect {
       case Term.Apply(Term.Select(_, retainMap(n: Name)), List(_: Term.PartialFunction)) =>
         ctx.replaceTree(n, "filterInPlace")
@@ -72,7 +72,7 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
         ).asPatch
     }.asPatch
 
-  def replaceSymbolicFold(ctx: RuleCtx) = 
+  def replaceSymbolicFold(ctx: RuleCtx): Patch = 
     ctx.tree.collect {
       case Term.Apply(ap @ Term.ApplyInfix(rhs, foldRightSymbol(_), _, List(lhs)), _) => 
         ctx.replaceTree(ap, s"$rhs.foldRight($lhs)")
@@ -81,7 +81,7 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
         ctx.replaceTree(ap, s"$rhs.foldLeft($lhs)")
     }.asPatch
 
-  def replaceToList(ctx: RuleCtx) =
+  def replaceToList(ctx: RuleCtx): Patch = 
     ctx.tree.collect {
       case iterator(t: Name) =>
         ctx.replaceTree(t, "iterator")
@@ -96,7 +96,7 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
         ).asPatch
     }.asPatch
 
-  def replaceTupleZipped(ctx: RuleCtx) =
+  def replaceTupleZipped(ctx: RuleCtx): Patch = 
     ctx.tree.collect {
       case tupleZipped(Term.Select(Term.Tuple(args), name)) =>
         val removeTokensPatch =
@@ -157,8 +157,6 @@ case class Scalacollectioncompat_newcollections(index: SemanticdbIndex)
     }.asPatch
 
   override def fix(ctx: RuleCtx): Patch = {
-    // println(ctx.index.database)
-
     replaceToList(ctx) +
       replaceSymbols(ctx) +
       replaceTupleZipped(ctx) +

scalafix/tests/src/test/scala/fix/Collectionstrawman_Tests.scala

@@ -1,14 +1,10 @@
 package fix
 
 import scala.meta._
-import scalafix.testkit._
 import scalafix._
+import scalafix.v0._
 
-class Collectionstrawman_Tests
-  extends SemanticRuleSuite(
-    SemanticdbIndex.load(Classpath(AbsolutePath(BuildInfo.inputClassdirectory))),
-    AbsolutePath(BuildInfo.inputSourceroot),
-    Seq(AbsolutePath(BuildInfo.outputSourceroot))
-  ) {
+class Scalafixplayground_Tests
+  extends scalafix.testkit.SemanticRuleSuite {
   runAllTests()
 }