Dotty VarArg pattern (#59)

build.sbt

@@ -369,7 +369,7 @@ def CiCommand(name: String)(commands: List[String]): Command =
  case (state, command) => command :: state
  }
  }
-def ci(command: String) = s"plz ${ciScalaVersion.get} $command"
+def ci(command: String) = s"plz ${ciScalaVersion.getOrElse("No CI_SCALA_VERSION defined")} $command"
 
 lazy val compilerOptions = Seq(
  "-deprecation",

readme/Rewrites.scalatex

@@ -139,6 +139,16 @@
  Such members already have a return type of @code{Unit} and sometimes this is unexpected.
  Adding an explicit return type makes it more obvious.
 
+ @sect(DottyVarArgPattern.toString)
+ @p
+ Replaces @code{@@} symbols in VarArg patterns with a colon (@code{:}). See @lnk{http://dotty.epfl.ch/docs/reference/changed/vararg-patterns.html}
+
+ @hl.scala
+ // before
+ case List(1, 2, xs @@ _*)
+ // after
+ case List(1, 2, xs : _*)
+
  @sect(NoAutoTupling.toString)
  @p
  Adds explicit tuples around argument lists where auto-tupling is occurring.

scalafix-core/src/main/scala/scalafix/rewrite/DottyVarArgPattern.scala

@@ -0,0 +1,20 @@
+package scalafix.rewrite
+
+import scala.meta._
+
+import scala.meta.tokens.Token
+import scalafix.Patch
+
+case object DottyVarArgPattern extends Rewrite {
+ override def rewrite(ctx: RewriteCtx): Patch = {
+ val patches = ctx.tree.collect {
+ case bind @ Pat.Bind(_, Pat.Arg.SeqWildcard()) =>
+ ctx.tokenList.leading(bind.tokens.last).collectFirst {
+ case tok @ Token.At() =>
+ ctx.replaceToken(tok, ":")
+ }.asPatch
+ }
+ patches.asPatch
+ }
+
+}

scalafix-core/src/main/scala/scalafix/rewrite/ScalafixRewrites.scala

@@ -8,7 +8,8 @@ object ScalafixRewrites {
  VolatileLazyVal,
  RemoveXmlLiterals,
  ExplicitUnit,
- NoValInForComprehension
+ NoValInForComprehension,
+ DottyVarArgPattern
  )
  def semantic(mirror: Mirror): List[Rewrite] = List(
  ExplicitReturnTypes(mirror),

scalafix-tests/input/src/main/scala/test/DottyVarArgPattern.scala

@@ -0,0 +1,12 @@
+/*
+rewrites = DottyVarArgPattern
+ */
+package test
+
+object DottyVarArgPattern {
+ List(1, 2, 3, 4) match {
+ case List(1, 2, xs @ _*) =>
+ }
+
+ val List(1, 2, x @ _*) = List(1, 2, 3, 4)
+}

scalafix-tests/output-dotty/src/main/scala/test/DottyVarArgPattern.scala

@@ -0,0 +1,9 @@
+package test
+
+object DottyVarArgPattern {
+ List(1, 2, 3, 4) match {
+ case List(1, 2, xs : _*) =>
+ }
+
+ val List(1, 2, x : _*) = List(1, 2, 3, 4)
+}