Merge pull request #140 from ckipp01/update/mill-scalafix-0.3.2

Update mill-scalafix to 0.3.2

build.sc

@@ -1,4 +1,4 @@
-import $ivy.`com.goyeau::mill-scalafix::0.3.1`
+import $ivy.`com.goyeau::mill-scalafix::0.3.2`
 import $ivy.`com.lihaoyi::mill-contrib-buildinfo:$MILL_VERSION`
 import $ivy.`de.tototec::de.tobiasroeser.mill.integrationtest::0.7.1`
 import $ivy.`io.chris-kipp::mill-ci-release::0.1.9`

domain/src/io/kipp/github/dependency/graph/domain/DependencyNode.scala

@@ -23,7 +23,7 @@ final case class DependencyNode(
 
   /** Returns true if both the relationship exists and is direct
     */
-  def isDirectDependency = relationship match {
+  def isDirectDependency: Boolean = relationship match {
     case None                                  => false
     case Some(DependencyRelationship.indirect) => false
     case Some(DependencyRelationship.direct)   => true

plugin/src/io/kipp/mill/github/dependency/graph/ModuleTrees.scala

@@ -161,7 +161,7 @@ final case class ModuleTrees(
     allDependencies.toMap
   }
 
-  def toManifest()(implicit ctx: mill.api.Ctx) = {
+  def toManifest()(implicit ctx: mill.api.Ctx): Manifest = {
     // NOTE: That this may seem odd when reading the spec that we have a
     // manifest per module basically, but we did check with the GitHub team and
     // they verified the manifests that we showed them.

plugin/src/io/kipp/mill/github/dependency/graph/Writers.scala

@@ -86,7 +86,7 @@ object Writers {
   }
 
   private implicit class MetadataJson(metadata: Map[String, String]) {
-    def toJsonValue = {
+    def toJsonValue: Obj = {
       ujson.Obj.from(
         metadata.map { case (key, value) =>
           (key, ujson.Str(value))