Auto-format imports

Author: japgolly

.github/workflows/ci.yml

@@ -32,16 +32,16 @@ jobs:
         run: ./prepareForScala210.sh
 
       - name: Build
-        run: sbt "++${{ matrix.scalaversion }}" package
+        run: sbt -DCI=1 "++${{ matrix.scalaversion }}" package
 
       - name: Test generate documentation
-        run: sbt "++${{ matrix.scalaversion }}" doc
+        run: sbt -DCI=1 "++${{ matrix.scalaversion }}" doc
 
       - name: Build examples
-        run: sbt "++${{ matrix.scalaversion }}" example/compile
+        run: sbt -DCI=1 "++${{ matrix.scalaversion }}" example/compile
 
       - name: Validate formatting
-        run: sbt "++${{ matrix.scalaversion }}" scalafmtCheck
+        run: sbt -DCI=1 "++${{ matrix.scalaversion }}" scalafmtCheck
 
       - name: Validate api report
         if: matrix.scalajsversion == '1.x' && matrix.scalaversion != '2.11.12'

.scalafix.conf

@@ -1,3 +1,20 @@
 rules = [
+  OrganizeImports,
+  RemoveUnused,
   GenerateApiReport,
 ]
+
+RemoveUnused {
+  imports  = false
+  privates = true
+  locals   = true
+}
+
+OrganizeImports {
+  expandRelative                             = true
+  groupedImports                             = Merge
+  groupExplicitlyImportedImplicitsSeparately = false
+  groups                                     = ["*"]
+  importSelectorsOrder                       = Ascii
+  removeUnused                               = true
+}

build.sbt

@@ -23,14 +23,28 @@ lazy val root = project
 name := "Scala.js DOM"
 
 ThisBuild / crossScalaVersions := {
-  if (scalaJSVersion.startsWith("1.")) Seq("2.12.10", "2.11.12", "2.13.1")
-  else Seq("2.12.10", "2.11.12", "2.10.7", "2.13.1")
+  if (scalaJSVersion.startsWith("1.")) Seq("2.11.12", "2.12.10","2.13.1")
+  else Seq("2.10.7", "2.11.12", "2.12.10", "2.13.1")
 }
-ThisBuild / scalaVersion := crossScalaVersions.value.head
+ThisBuild / scalaVersion := crossScalaVersions.value.find(_.startsWith("2.13.")).get
+
+val inCI = Option(System.getenv("CI")).exists(_.contains("1"))
 
 val commonSettings = Seq(
   organization := "org.scala-js",
-  scalacOptions ++= Seq("-deprecation", "-feature", "-Xfatal-warnings")
+  scalacOptions ++= Seq(
+    "-deprecation",
+    "-feature",
+  ),
+  scalacOptions ++= (if (!inCI) Seq.empty else Seq(
+    "-Xfatal-warnings",
+  )),
+  scalacOptions ++= (CrossVersion.partialVersion(scalaVersion.value) match {
+    case Some((2, 11)) => "-Ywarn-unused-import" :: Nil
+    case Some((2, 12)) => "-Ywarn-unused:imports,patvars,locals,implicits" :: Nil
+    case Some((2, 13)) => "-Wunused:imports,patvars,locals,implicits" :: Nil
+    case _             => Nil
+  }),
 )
 
 val noPublishSettings = Seq(

scalafix.sbt

@@ -8,3 +8,7 @@ ThisBuild / scalacOptions ++= {
   else
     Nil
 }
+
+ThisBuild / scalafixDependencies ++= Seq(
+  "com.github.liancheng" %% "organize-imports" % "0.5.0"
+)

src/main/scala-new-collections/org/scalajs/dom/ext/NamedNodeMapMap.scala

@@ -1,8 +1,7 @@
 package org.scalajs.dom.ext
 
-import scala.collection.mutable
-
 import org.scalajs.dom._
+import scala.collection.mutable
 
 class NamedNodeMapMap private[ext] (namedNodeMap: NamedNodeMap)
     extends mutable.Map[String, Attr] {

src/main/scala-old-collections/org/scalajs/dom/ext/NamedNodeMapMap.scala

@@ -1,8 +1,7 @@
 package org.scalajs.dom.ext
 
-import scala.collection.mutable
-
 import org.scalajs.dom._
+import scala.collection.mutable
 
 class NamedNodeMapMap private[ext] (namedNodeMap: NamedNodeMap)
     extends mutable.Map[String, Attr] {

src/main/scala/org/scalajs/dom/crypto/package.scala

@@ -1,11 +1,10 @@
 package org.scalajs.dom
 
+import scala.language.implicitConversions
 import scala.scalajs.js
-import scala.scalajs.js.typedarray.{ArrayBufferView, ArrayBuffer, Uint8Array}
+import scala.scalajs.js.typedarray.{ArrayBuffer, ArrayBufferView, Uint8Array}
 import scala.scalajs.js.|
 
-import scala.language.implicitConversions
-
 package object crypto {
   @inline def crypto = GlobalCrypto.crypto
 

src/main/scala/org/scalajs/dom/experimental/AbortController.scala

@@ -1,7 +1,6 @@
 package org.scalajs.dom.experimental
 
 import org.scalajs.dom.raw.EventTarget
-
 import scala.scalajs.js
 import scala.scalajs.js.annotation.JSGlobal
 

src/main/scala/org/scalajs/dom/experimental/Fetch.scala

@@ -2,7 +2,6 @@ package org.scalajs.dom.experimental
 
 import org.scalajs.dom.Blob
 import org.scalajs.dom.raw.FormData
-
 import scala.scalajs.js
 import scala.scalajs.js.annotation._
 import scala.scalajs.js.typedarray.{ArrayBuffer, Uint8Array}

src/main/scala/org/scalajs/dom/experimental/FileReaderSync.scala

@@ -1,7 +1,6 @@
 package org.scalajs.dom.experimental
 
 import org.scalajs.dom.raw.Blob
-
 import scala.scalajs.js
 import scala.scalajs.js.annotation._
 import scala.scalajs.js.typedarray.ArrayBuffer