fix: ensure discover macro is in the same file

closes #96
ckipp01 · Jun 4, 2023 · 2132b48 · 2132b48
1 parent 1464f8b
commit 2132b48
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/plugin/src-mill0.10/io/kipp/mill/github/dependency/graph/Discover.scala b/plugin/src-mill0.10/io/kipp/mill/github/dependency/graph/Discover.scala
@@ -5,6 +5,4 @@ import mill.main.EvaluatorScopt
 private[graph] object Discover {
   implicit def millScoptEvaluatorReads[A]: EvaluatorScopt[A] =
     new EvaluatorScopt[A]()
-
-  def apply[T] = mill.define.Discover[T]
 }
diff --git a/plugin/src-mill0.11/io/kipp/mill/github/dependency/graph/Discover.scala b/plugin/src-mill0.11/io/kipp/mill/github/dependency/graph/Discover.scala
@@ -1,5 +1,6 @@
 package io.kipp.mill.github.dependency.graph
 
 private[graph] object Discover {
-  def apply[T] = mill.define.Discover[T]
+  implicit def millEvaluatorTokenReader =
+    mill.main.TokenReaders.millEvaluatorTokenReader
 }
diff --git a/plugin/src/io/kipp/mill/github/dependency/graph/Graph.scala b/plugin/src/io/kipp/mill/github/dependency/graph/Graph.scala
@@ -6,6 +6,10 @@ import mill.define.Command
 import mill.define.ExternalModule
 import mill.eval.Evaluator
 
+import scala.annotation.nowarn
+
+// In here for the Discover import
+@nowarn("msg=Unused import")
 object Graph extends ExternalModule {
 
   import Writers._
@@ -26,5 +30,7 @@ object Graph extends ExternalModule {
       manifests
     }
 
-  lazy val millDiscover: mill.define.Discover[this.type] = Discover[this.type]
+  import Discover._
+  lazy val millDiscover: mill.define.Discover[this.type] =
+    mill.define.Discover[this.type]
 }