Cleanup

Author: lefou

core/api/daemon/src/mill/api/daemon/internal/OptsApi.scala

@@ -6,7 +6,6 @@ trait OptsApi {
 }
 
 trait OptGroupApi {
-  def startsWithArg(prefix: String): Boolean
 }
 
 trait OptApi {

core/api/src/mill/api/opt/Opt.scala

@@ -5,7 +5,6 @@ import mill.api.daemon.internal.OptApi
 import mill.api.JsonFormatters.given
 
 import scala.annotation.targetName
-import scala.language.implicitConversions
 
 type OptTypes = (String | os.Path)
 

core/api/src/mill/api/opt/OptGroup.scala

@@ -12,36 +12,18 @@ case class OptGroup private (value: Seq[Opt]) extends OptGroupApi
     derives upickle.ReadWriter {
   override def toString(): String = value.mkString("(", ", ", ")")
 
-//  /**
-//   * Unify multiple options into one option.
-//   * Same as `mkString(" ")`` on a options list of type `Seq[String].
-//   */
-//  def mkOpt(sep: String = " "): Opt = Opt(value*)
-
   def isEmpty: Boolean = value.isEmpty
 
   def containsPaths: Boolean = value.exists(_.containsPaths)
 
   def head: Opt = value.head
-  @deprecated
-  override def startsWithArg(prefix: String): Boolean = internalStartsWith(prefix)
-  @deprecated
-  def startsWithArg(prefix: os.Path): Boolean = internalStartsWith(prefix)
-  @deprecated
-  private inline def internalStartsWith(prefix: (String | os.Path)): Boolean =
-    value.headOption.exists {
-      head =>
-        (head, prefix) match {
-          case (h: String, s: String) => h == s
-          case (h: os.Path, p: os.Path) => h == p
-          case _ => false
-        }
-    }
+
+  def toStringSeq: Seq[String] = value.map(_.toString)
 }
 
 object OptGroup {
   @targetName("applyUnion")
-  def apply(value: (Opt | Seq[Opt])*): OptGroup = OptGroup(value.flatMap {
+  def apply(value: (Opt | Seq[Opt])*): OptGroup = new OptGroup(value.flatMap {
     case a: Opt => Seq(a)
     case s: Seq[Opt] => s
   })
@@ -65,4 +47,3 @@ object OptGroup {
     OptGroup(s.flatMap(f(_).value))
 
 }
-