iterator() -> iterator

Author: lrytz

.travis.yml

@@ -3,7 +3,7 @@ language: scala
 scala:
   - 2.11.12
   - 2.12.6
-  - 2.13.0-pre-abf21b9
+  - 2.13.0-pre-a002630
 jdk:
   - oraclejdk8
 env:
@@ -21,9 +21,9 @@ env:
 
 matrix:
   exclude:
-    - scala: 2.13.0-pre-abf21b9
+    - scala: 2.13.0-pre-a002630
       env: SCALAJS_VERSION=0.6.22
-    - scala: 2.13.0-pre-abf21b9
+    - scala: 2.13.0-pre-a002630
       env: SCALAJS_VERSION=1.0.0-M3
   include:
     - scala: 2.12.6

CONTRIBUTING.md

@@ -20,7 +20,7 @@ the strawman:
 
 ~~~ diff
 - xs.toIterator
-+ xs.iterator()
++ xs.iterator
 ~~~
 
 ### Add a missing test case
@@ -45,7 +45,7 @@ class toIteratorVsIterator(xs: Iterable[Int]) {
 import strawman.collection.Iterable
 
 class toIteratorVsIterator(xs: Iterable[Int]) {
-  xs.iterator()
+  xs.iterator
 }
 ~~~
 

build.sbt

@@ -3,7 +3,7 @@ import ScalaModulePlugin._
 
 inThisBuild(Seq(
   resolvers += "scala-pr" at "https://scala-ci.typesafe.com/artifactory/scala-integration/",
-  crossScalaVersions := Seq("2.12.6", "2.13.0-pre-abf21b9", "2.11.12")
+  crossScalaVersions := Seq("2.12.6", "2.13.0-pre-a002630", "2.11.12")
 ))
 
 lazy val `scala-collection-compat` = crossProject(JSPlatform, JVMPlatform)

scalafix/build.sbt

@@ -23,7 +23,7 @@ lazy val input = project
 lazy val output = project
   .settings(
     resolvers += "scala-pr" at "https://scala-ci.typesafe.com/artifactory/scala-integration/",
-    scalaVersion := "2.13.0-pre-abf21b9"
+    scalaVersion := "2.13.0-pre-a002630"
   )
 
 lazy val tests = project

scalafix/output/src/main/scala/fix/TraversableSrc.scala

@@ -4,7 +4,7 @@ object TraversableSrc {
   def foo(xs: Iterable[(Int, String)], ys: List[Int]): Unit = {
     xs.to(List)
     xs.to(Set)
-    xs.iterator()
-    ys.iterator()
+    xs.iterator
+    ys.iterator
   }
 }

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

@@ -23,7 +23,6 @@ case class Scalacollectioncompat_NewCollections(index: SemanticdbIndex)
     Symbol("_root_.scala.collection.TraversableLike.to.")
   )
   val iterator = SymbolMatcher.normalized(
-    Symbol("_root_.scala.collection.LinearSeqLike.iterator."),
     Symbol("_root_.scala.collection.TraversableLike.toIterator.")
   )
   val tupleZipped = SymbolMatcher.normalized(
@@ -34,7 +33,7 @@ case class Scalacollectioncompat_NewCollections(index: SemanticdbIndex)
   def replaceToList(ctx: RuleCtx) =
     ctx.tree.collect {
       case iterator(t: Name) =>
-        ctx.replaceTree(t, "iterator()")
+        ctx.replaceTree(t, "iterator")
       case toTpe(n: Name) =>
         (for {
           name <- n.tokens.lastOption