Clean unused imports (#2776)

Pull request: #2776

bsp/src/mill/bsp/BspContext.scala

@@ -1,7 +1,6 @@
 package mill.bsp
 
-import mill.api.{DummyInputStream, Logger, SystemStreams, internal}
-import mill.eval.Evaluator
+import mill.api.{DummyInputStream, Logger, SystemStreams}
 
 import java.io.PrintStream
 import scala.util.control.NonFatal

bsp/src/mill/bsp/BspWorker.scala

@@ -1,7 +1,6 @@
 package mill.bsp
 
-import mill.api.{Ctx, Logger, SystemStreams, internal}
-import mill.eval.Evaluator
+import mill.api.{Ctx, Logger, SystemStreams}
 import os.Path
 
 import java.io.PrintStream

contrib/versionfile/src/mill/contrib/versionfile/VersionFileModule.scala

@@ -64,7 +64,6 @@ trait VersionFileModule extends Module {
       case snapshot: Version.Snapshot => s"Setting next version to $version"
     }
 
-  import upickle.core._
   import upickle.default._
 
   implicit val shellableReadWriter: ReadWriter[os.Shellable] =

main/api/src/mill/api/IO.scala

@@ -1,7 +1,5 @@
 package mill.api
 
-import java.io.{InputStream, OutputStream}
-
 /**
  * Misc IO utilities, eventually probably should be pushed upstream into
  * ammonite-ops

main/api/src/mill/api/PathRef.scala

@@ -6,8 +6,6 @@ import java.util.concurrent.ConcurrentHashMap
 import scala.util.{DynamicVariable, Using}
 import upickle.default.{ReadWriter => RW}
 
-import scala.runtime.ScalaRunTime
-
 /**
  * A wrapper around `os.Path` that calculates it's hashcode based
  * on the contents of the filesystem underneath it. Used to ensure filesystem

main/define/src/mill/define/Applicative.scala

@@ -2,8 +2,7 @@ package mill.define
 
 import mill.api.internal
 
-import scala.annotation.{StaticAnnotation, compileTimeOnly}
-import scala.language.higherKinds
+import scala.annotation.compileTimeOnly
 import scala.reflect.macros.blackbox.Context
 
 /**

main/define/src/mill/define/Cross.scala

@@ -3,7 +3,6 @@ package mill.define
 import mill.api.Lazy
 
 import language.experimental.macros
-import scala.collection.SeqView
 import scala.reflect.ClassTag
 import scala.reflect.macros.blackbox
 

main/define/src/mill/define/Ctx.scala

@@ -1,7 +1,5 @@
 package mill.define
 
-import os.Path
-
 import scala.annotation.implicitNotFound
 
 /**

main/define/src/mill/define/Module.scala

@@ -2,7 +2,6 @@ package mill.define
 
 import mill.api.internal
 
-import scala.language.experimental.macros
 import scala.reflect.ClassTag
 
 /**

main/define/src/mill/define/Task.scala

@@ -2,7 +2,7 @@ package mill.define
 
 import mill.api.{CompileProblemReporter, Logger, PathRef, Result, TestReporter}
 import mill.define.Applicative.Applyable
-import upickle.default.{ReadWriter => RW, Reader => R, Writer => W}
+import upickle.default.{ReadWriter => RW, Writer => W}
 
 import scala.language.experimental.macros
 import scala.reflect.macros.blackbox.Context

main/eval/src/mill/eval/Evaluator.scala

@@ -2,7 +2,7 @@ package mill.eval
 
 import mill.api.{CompileProblemReporter, DummyTestReporter, Result, TestReporter, Val}
 import mill.api.Strict.Agg
-import mill.define.{BaseModule, NamedTask, Segments, Task}
+import mill.define.{BaseModule, Segments, Task}
 import mill.eval.Evaluator.{Results, formatFailing}
 import mill.util.{ColorLogger, MultiBiMap}
 

main/eval/src/mill/eval/EvaluatorPaths.scala

@@ -1,7 +1,7 @@
 package mill.eval
 
 import mill.api.internal
-import mill.define.{NamedTask, Segment, Segments, Target}
+import mill.define.{NamedTask, Segment, Segments}
 
 case class EvaluatorPaths private (dest: os.Path, meta: os.Path, log: os.Path) {
   private def copy(dest: os.Path = dest, meta: os.Path = meta, log: os.Path = log): EvaluatorPaths =

main/eval/test/src/mill/eval/CrossTests.scala

@@ -1,6 +1,5 @@
 package mill.eval
 
-import mill.define.Discover
 import mill.util.{TestEvaluator, TestGraphs}
 import mill.util.TestGraphs.{
   crossResolved,
@@ -12,6 +11,7 @@ import mill.util.TestGraphs.{
   crossExtension
 }
 import utest._
+
 object CrossTests extends TestSuite {
   val tests = Tests {
     "singleCross" - {

main/resolve/src/mill/resolve/ResolveNotFoundHandler.scala

@@ -1,7 +1,6 @@
 package mill.resolve
 
-import mill.api.internal
-import mill.define.{BaseModule, Discover, Segment, Segments}
+import mill.define.{Segment, Segments}
 import mill.resolve.ResolveCore.Resolved
 
 /**

main/resolve/src/mill/resolve/SimpleTaskTokenReader.scala

@@ -1,7 +1,7 @@
 package mill.resolve
 
 import mainargs.TokensReader
-import mill.define.{Target, Task, Args}
+import mill.define.{Target, Task}
 
 /**
  * Transparently handle `Task[T]` like simple `T` but lift the result into a T.task.

main/src/mill/main/MainModule.scala

@@ -8,7 +8,7 @@ import mill.define.{Command, NamedTask, Segments, Task}
 import mill.eval.{Evaluator, EvaluatorPaths, Terminal}
 import mill.resolve.{Resolve, SelectMode}
 import mill.resolve.SelectMode.Separated
-import mill.util.{PrintLogger, Watchable}
+import mill.util.Watchable
 import pprint.{Renderer, Tree, Truncated}
 
 import scala.collection.mutable

main/src/mill/main/RootModule.scala

@@ -1,10 +1,7 @@
 package mill.main
 
-import mill.api.{PathRef, internal}
+import mill.api.internal
 import mill.define.{Caller, Discover, Segments}
-import TokenReaders._
-
-import scala.collection.mutable
 
 /**
  * Used to mark a module in your `build.sc` as a top-level module, so it's

main/test/src/mill/UTestFramework.scala

@@ -1,12 +1,10 @@
 package mill
-import utest.framework.{Result, StackMarker}
-import utest.ufansi.Str
 
 class UTestFramework extends utest.runner.Framework {
-  override def exceptionStackFrameHighlighter(s: StackTraceElement) = {
+  override def exceptionStackFrameHighlighter(s: StackTraceElement): Boolean = {
     s.getClassName.startsWith("mill.")
   }
-  override def setup() = {
+  override def setup(): Unit = {
 
     os.remove.all(os.pwd / "target" / "workspace")
   }

main/test/src/mill/util/TestEvaluator.scala

@@ -4,10 +4,12 @@ import java.io.{InputStream, PrintStream}
 import mill.eval.Evaluator
 import utest.framework.TestPath
 
-import language.experimental.macros
-import mill.api.{DummyInputStream, Result}
+import mill.api.DummyInputStream
+
 object TestEvaluator {
-  def static(module: => TestUtil.BaseModule)(implicit fullName: sourcecode.FullName) = {
+  def static(module: => TestUtil.BaseModule)(implicit
+      fullName: sourcecode.FullName
+  ): TestEvaluator = {
     new TestEvaluator(module)(fullName, TestPath(Nil))
   }
 }

main/testkit/src/mill/testkit/MillTestkit.scala

@@ -11,8 +11,6 @@ import mill.eval.Evaluator
 import mill.resolve.{Resolve, SelectMode}
 import mill.util.PrintLogger
 
-import language.experimental.macros
-
 trait MillTestKit {
 
   def defaultTargetDir: os.Path =

main/util/src/mill/util/Classpath.scala

@@ -2,12 +2,8 @@ package mill.util
 
 import java.io.File
 import java.net.URL
-import java.nio.file.{Path, Paths}
-import java.util.zip.{ZipFile, ZipInputStream}
-import mill.java9rtexport.Export
 
 import scala.collection.mutable
-import scala.util.control.NonFatal
 
 /**
  * Loads the jars that make up the classpath of the scala-js-fiddle

main/util/src/mill/util/Jvm.scala

@@ -1,22 +1,15 @@
 package mill.util
 
-import mill.api.BuildInfo
 import mill.api.Loose.Agg
 import mill.api._
 import mill.main.client.InputPumper
 import os.SubProcess
-import upickle.default.{ReadWriter => RW}
 
 import java.io._
 import java.lang.reflect.Modifier
-import java.net.URI
 import java.nio.file.attribute.PosixFilePermission
-import java.nio.file.{FileSystems, Files, StandardOpenOption}
-import java.util.Collections
-import java.util.jar.{Attributes, JarFile, Manifest}
-import scala.jdk.CollectionConverters._
+import java.nio.file.Files
 import scala.util.Properties.isWin
-import scala.util.Using
 
 object Jvm extends CoursierSupport {
 

main/util/src/mill/util/PrintLogger.scala

@@ -1,9 +1,7 @@
 package mill.util
 
 import java.io._
-import java.nio.file.{Files, StandardOpenOption}
-import scala.util.DynamicVariable
-import mill.api.{Logger, SystemStreams}
+import mill.api.SystemStreams
 
 class PrintLogger(
     override val colored: Boolean,

main/util/src/mill/util/Util.scala

@@ -2,7 +2,7 @@ package mill.util
 
 import coursier.Repository
 import mill.api.Loose.Agg
-import mill.api.{BuildInfo, Ctx, IO, Loose, PathRef, Result}
+import mill.api.{BuildInfo, Ctx, IO, PathRef, Result}
 
 object Util {
 

runner/src/mill/runner/MillMain.scala

@@ -9,7 +9,7 @@ import mill.java9rtexport.Export
 import mill.api.{DummyInputStream, internal}
 import mill.api.SystemStreams
 import mill.bsp.{BspContext, BspServerResult}
-import mill.util.{PrintLogger, Util}
+import mill.util.PrintLogger
 
 @internal
 object MillMain {

runner/src/mill/runner/RunnerState.scala

@@ -1,7 +1,7 @@
 package mill.runner
 
 import mill.api.{PathRef, Val, internal}
-import mill.define.{BaseModule, Segments}
+import mill.define.Segments
 import mill.util.Watchable
 import upickle.default.{ReadWriter, macroRW}
 import mill.api.JsonFormatters._

scalajslib/src/mill/scalajslib/ScalaJSModule.scala

@@ -13,8 +13,6 @@ import mill.scalajslib.internal.ScalaJSUtils.getReportMainFilePathRef
 import mill.scalajslib.worker.{ScalaJSWorker, ScalaJSWorkerExternalModule}
 import mill.scalalib.bsp.{ScalaBuildTarget, ScalaPlatform}
 
-import scala.jdk.CollectionConverters._
-
 trait ScalaJSModule extends scalalib.ScalaModule { outer =>
 
   def scalaJSVersion: T[String]

scalajslib/test/src/mill/scalajslib/SourceMapTests.scala

@@ -3,7 +3,6 @@ package mill.scalajslib
 import mill.define.Discover
 import mill.util.{TestEvaluator, TestUtil}
 import utest._
-import mill.define.Target
 
 object SourceMapTests extends TestSuite {
   val workspacePath = TestUtil.getOutPathStatic() / "source-map"

scalajslib/worker/1/src/mill/scalajslib/worker/jsenv/Phantom.scala

@@ -1,7 +1,6 @@
 package mill.scalajslib.worker.jsenv
 
 import mill.scalajslib.worker.api._
-import org.scalajs.jsenv.nodejs.NodeJSEnv.SourceMap
 
 object Phantom {
   def apply(config: JsEnvConfig.Phantom) =

scalalib/src/mill/scalalib/Dependency.scala

@@ -3,7 +3,6 @@ package mill.scalalib
 import mill.T
 import mill.define.{Command, Discover, ExternalModule}
 import mill.eval.Evaluator
-import mill.main.EvaluatorTokenReader
 import mill.scalalib.dependency.DependencyUpdatesImpl
 import mill.scalalib.dependency.updates.ModuleDependenciesUpdates
 

scalalib/src/mill/scalalib/JsonFormatters.scala

@@ -2,8 +2,6 @@ package mill.scalalib
 
 import upickle.default.{ReadWriter => RW}
 
-import scala.reflect.ClassTag
-
 trait JsonFormatters {
   implicit lazy val publicationFormat: RW[coursier.core.Publication] = upickle.default.macroRW
   implicit lazy val extensionFormat: RW[coursier.core.Extension] = upickle.default.macroRW

scalalib/src/mill/scalalib/Lib.scala

@@ -2,7 +2,7 @@ package mill
 package scalalib
 
 import coursier.util.Task
-import coursier.{Dependency, LocalRepositories, Repositories, Repository, Resolution}
+import coursier.{Dependency, Repository, Resolution}
 import mill.api.{Ctx, Loose, PathRef, Result}
 import mill.main.BuildInfo
 import mill.util.Util

scalalib/src/mill/scalalib/PublishModule.scala

@@ -3,10 +3,8 @@ package scalalib
 
 import mill.define.{Command, ExternalModule, Target, Task}
 import mill.api.{JarManifest, PathRef, Result}
-import mill.main.Tasks
-import mill.util.Jvm
 import mill.scalalib.PublishModule.checkSonatypeCreds
-import mill.scalalib.publish.{Artifact, SonatypePublisher, VersionScheme}
+import mill.scalalib.publish.{Artifact, SonatypePublisher}
 
 /**
  * Configuration necessary for publishing a Scala module to Maven Central or similar

scalalib/src/mill/scalalib/ScalaModule.scala

@@ -9,7 +9,6 @@ import mill.util.Jvm.createJar
 import mill.api.Loose.Agg
 import mill.scalalib.api.{CompilationResult, Versions, ZincWorkerUtil}
 
-import scala.jdk.CollectionConverters._
 import mainargs.Flag
 import mill.scalalib.bsp.{BspBuildTarget, BspModule, ScalaBuildTarget, ScalaPlatform}
 import mill.scalalib.dependency.versions.{ValidVersion, Version}

scalalib/src/mill/scalalib/SemanticDbJavaModule.scala

@@ -1,7 +1,7 @@
 package mill.scalalib
 
 import mill.api.{PathRef, Result, experimental}
-import mill.define.{ModuleRef, Target, Task}
+import mill.define.{ModuleRef, Target}
 import mill.main.BuildInfo
 import mill.scalalib.api.{CompilationResult, Versions, ZincWorkerUtil}
 import mill.scalalib.bsp.BspBuildTarget

scalalib/src/mill/scalalib/dependency/updates/ModuleDependenciesUpdates.scala

@@ -1,6 +1,5 @@
 package mill.scalalib.dependency.updates
 
-import mill.scalalib.JavaModule
 import mill.scalalib.dependency.versions.Version
 
 import scala.collection.SortedSet

scalalib/src/mill/scalalib/dependency/versions/VersionsFinder.scala

@@ -1,12 +1,10 @@
 package mill.scalalib.dependency.versions
 
-import scala.reflect.ClassTag
-
 import coursier.Dependency
-import mill.define.{BaseModule, Task}
+import mill.define.{BaseModule}
 import mill.eval.Evaluator
 import mill.scalalib.dependency.metadata.MetadataLoaderFactory
-import mill.scalalib.{Dep, JavaModule, Lib}
+import mill.scalalib.{JavaModule, Lib}
 import mill.api.Ctx.{Home, Log}
 import mill.T
 

scalalib/src/mill/scalalib/internal/JavaModuleUtils.scala

@@ -1,6 +1,6 @@
 package mill.scalalib.internal
 
-import mill.define.{Module, Segments}
+import mill.define.Module
 import mill.scalalib.JavaModule
 
 @mill.api.internal

scalalib/src/mill/scalalib/publish/SonatypeHttpApi.scala

@@ -4,7 +4,6 @@ import java.util.Base64
 
 import scala.annotation.tailrec
 import scala.concurrent.duration._
-import scala.util.control.NonFatal
 
 import mill.main.BuildInfo
 import requests.BaseSession

scalalib/src/mill/scalalib/publish/SonatypePublisher.scala

@@ -5,7 +5,6 @@ import java.security.MessageDigest
 
 import mill.api.Logger
 import mill.util.Jvm
-import os.Shellable
 
 class SonatypePublisher(
     uri: String,