scalafmt: add formatter to the build

.git-blame-ignore-revs

@@ -0,0 +1,2 @@
+# initial format for 3.10.7
+bee6f0547cf036a8007becefd500328e38b5d5e7

.github/workflows/ci.yml

@@ -8,6 +8,19 @@ on:
   pull_request:
 
 jobs:
+  formatting:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v6
+      with:
+        fetch-depth: 0
+        submodules: false
+    - uses: coursier/setup-action@v2.0
+      with:
+        jvm: 8
+        apps: sbt
+    - run: sbt scalafmtCheckAll
+
   test:
     name: ${{ matrix.os }} ${{ matrix.PLUGIN }} group ${{ matrix.TEST_GROUP }}
     runs-on: ${{ matrix.os }}

.scalafmt.conf

@@ -0,0 +1,25 @@
+version = 3.10.7
+align {
+  preset = none
+}
+docstrings {
+  style = Asterisk
+  blankFirstLine = keep
+  wrap = keep
+}
+indent {
+  defnSite = 2
+  infix.exemptScope = []
+}
+maxColumn = 120
+newlines {
+  source = keep
+  avoidForSimpleOverflow = all
+  inInterpolation = avoid
+}
+rewrite {
+  trailingCommas.style = keep
+}
+runner {
+  dialect = scala213
+}

modules/definitions/src/main/scala/lmcoursier/CoursierConfiguration.scala

@@ -5,8 +5,12 @@ import dataclass.{data, since}
 import coursier.cache.CacheDefaults
 import coursier.params.rule.{Rule, RuleResolution}
 import lmcoursier.credentials.Credentials
-import lmcoursier.definitions.{Authentication, CacheLogger, CachePolicy, FromCoursier, Module, ModuleMatchers, Project, Reconciliation, Strict}
-import sbt.librarymanagement.{CrossVersion, InclExclRule, ModuleDescriptorConfiguration, ModuleID, ModuleInfo, Resolver, UpdateConfiguration}
+import lmcoursier.definitions.{
+  Authentication, CacheLogger, CachePolicy, FromCoursier, Module, ModuleMatchers, Project, Reconciliation, Strict
+}
+import sbt.librarymanagement.{
+  CrossVersion, InclExclRule, ModuleDescriptorConfiguration, ModuleID, ModuleInfo, Resolver, UpdateConfiguration
+}
 import xsbti.Logger
 
 import scala.concurrent.duration.{Duration, FiniteDuration}

modules/definitions/src/main/scala/lmcoursier/definitions/Authentication.scala

@@ -8,7 +8,7 @@ import dataclass._
   optional: Boolean = false,
   realmOpt: Option[String] = None,
   @since("1.0")
-  headers: Seq[(String,String)] = Nil,
+  headers: Seq[(String, String)] = Nil,
   @since("1.1")
   httpsOnly: Boolean = true,
   @since("1.2")

modules/lm-coursier/src/main/scala/lmcoursier/CoursierDependencyResolution.scala

@@ -9,7 +9,10 @@ import coursier.cache.{CacheDefaults, CachePolicy}
 import coursier.util.Artifact
 import coursier.internal.Typelevel
 import lmcoursier.definitions.ToCoursier
-import lmcoursier.internal.{ArtifactsParams, ArtifactsRun, CoursierModuleDescriptor, InterProjectRepository, ResolutionParams, ResolutionRun, Resolvers, SbtBootJars, UpdateParams, UpdateRun}
+import lmcoursier.internal.{
+  ArtifactsParams, ArtifactsRun, CoursierModuleDescriptor, InterProjectRepository, ResolutionParams, ResolutionRun,
+  Resolvers, SbtBootJars, UpdateParams, UpdateRun
+}
 import lmcoursier.syntax._
 import sbt.internal.librarymanagement.IvySbt
 import sbt.librarymanagement._
@@ -71,7 +74,7 @@ class CoursierDependencyResolution(
         ModuleID("lmcoursier", "lmcoursier", "0.1.0"),
         ModuleInfo("protocol-handler")
       )
-      .withDependencies(conf0.protocolHandlerDependencies.toVector)
+        .withDependencies(conf0.protocolHandlerDependencies.toVector)
 
     val reportOrUnresolved = resolution.update(moduleDescriptor(fakeModule), configuration, uwconfig, log)
 
@@ -345,14 +348,14 @@ class CoursierDependencyResolution(
     }
 
     if (otherErrors.isEmpty) {
-        val r = new ResolveException(
-          downloadErrors.map(_.getMessage),
-          downloadErrors.map { err =>
-            ModuleID(err.module.organization.value, err.module.name.value, err.version)
-              .withExtraAttributes(err.module.attributes)
-          }
-        )
-        UnresolvedWarning(r, uwconfig)
+      val r = new ResolveException(
+        downloadErrors.map(_.getMessage),
+        downloadErrors.map { err =>
+          ModuleID(err.module.organization.value, err.module.name.value, err.version)
+            .withExtraAttributes(err.module.attributes)
+        }
+      )
+      UnresolvedWarning(r, uwconfig)
     } else
       throw ex
   }
@@ -362,8 +365,10 @@ object CoursierDependencyResolution {
   def apply(configuration: CoursierConfiguration): DependencyResolution =
     DependencyResolution(new CoursierDependencyResolution(configuration))
 
-  def apply(configuration: CoursierConfiguration,
-            protocolHandlerConfiguration: Option[CoursierConfiguration]): DependencyResolution =
+  def apply(
+    configuration: CoursierConfiguration,
+    protocolHandlerConfiguration: Option[CoursierConfiguration]
+  ): DependencyResolution =
     DependencyResolution(
       new CoursierDependencyResolution(
         configuration,

modules/lm-coursier/src/main/scala/lmcoursier/FromSbt.scala

@@ -1,7 +1,9 @@
 package lmcoursier
 
 import coursier.ivy.IvyXml.{mappings => ivyXmlMappings}
-import lmcoursier.definitions.{Classifier, Configuration, Dependency, Extension, Info, Module, ModuleName, Organization, Project, Publication, Type}
+import lmcoursier.definitions.{
+  Classifier, Configuration, Dependency, Extension, Info, Module, ModuleName, Organization, Project, Publication, Type
+}
 import sbt.internal.librarymanagement.mavenint.SbtPomExtraProperties
 import sbt.librarymanagement.{Configuration => _, MavenRepository => _, _}
 
@@ -43,7 +45,8 @@ object FromSbt {
 
     val fullName = sbtModuleIdName(module, scalaVersion, scalaBinaryVersion, optionalCrossVer)
 
-    val module0 = Module(Organization(module.organization), ModuleName(fullName), attributes(module.extraDependencyAttributes))
+    val module0 =
+      Module(Organization(module.organization), ModuleName(fullName), attributes(module.extraDependencyAttributes))
     val version = module.revision
 
     (module0, version)

modules/lm-coursier/src/main/scala/lmcoursier/Inputs.scala

@@ -15,7 +15,8 @@ object Inputs {
         Configuration(from.value) -> Configuration(to.value)
     }
 
-  def configExtendsSeq(configurations: Seq[sbt.librarymanagement.Configuration]): Seq[(Configuration, Seq[Configuration])] =
+  def configExtendsSeq(configurations: Seq[sbt.librarymanagement.Configuration])
+    : Seq[(Configuration, Seq[Configuration])] =
     configurations
       .map(cfg => Configuration(cfg.name) -> cfg.extendsConfigs.map(c => Configuration(c.name)))
 
@@ -26,10 +27,14 @@ object Inputs {
       .toMap
 
   @deprecated("Use coursierConfigurationsMap instead", "2.0.0-RC6-5")
-  def coursierConfigurations(configurations: Seq[sbt.librarymanagement.Configuration], shadedConfigOpt: Option[String] = None): Map[Configuration, Set[Configuration]] =
+  def coursierConfigurations(
+    configurations: Seq[sbt.librarymanagement.Configuration],
+    shadedConfigOpt: Option[String] = None
+  ): Map[Configuration, Set[Configuration]] =
     coursierConfigurationsMap(configurations)
 
-  def coursierConfigurationsMap(configurations: Seq[sbt.librarymanagement.Configuration]): Map[Configuration, Set[Configuration]] = {
+  def coursierConfigurationsMap(configurations: Seq[sbt.librarymanagement.Configuration])
+    : Map[Configuration, Set[Configuration]] = {
 
     val configs0 = configExtendsSeq(configurations).toMap
 

modules/lm-coursier/src/main/scala/lmcoursier/IvyXml.scala

@@ -80,7 +80,9 @@ object IvyXml {
 
     val publicationElems = publications.map {
       case (pub, configs) =>
-        val n = <artifact name={pub.name} type={pub.`type`.value} ext={pub.ext.value} conf={configs.map(_.value).mkString(",")} />
+        val n = <artifact name={pub.name} type={pub.`type`.value} ext={pub.ext.value} conf={
+          configs.map(_.value).mkString(",")
+        } />
 
         if (pub.classifier.value.nonEmpty)
           n % <x e:classifier={pub.classifier.value} />.attributes
@@ -95,7 +97,9 @@ object IvyXml {
             <exclude org={org.value} module={name.value} name="*" type="*" ext="*" conf="" matcher="exact"/>
         }
 
-        val n = <dependency org={dep.module.organization.value} name={dep.module.name.value} rev={dep.version} conf={s"${conf.value}->${dep.configuration.value}"}>
+        val n = <dependency org={dep.module.organization.value} name={dep.module.name.value} rev={dep.version} conf={
+          s"${conf.value}->${dep.configuration.value}"
+        }>
           {excludes}
         </dependency>
 

modules/lm-coursier/src/main/scala/lmcoursier/credentials/Credentials.scala

@@ -12,7 +12,13 @@ object Credentials {
     DirectCredentials(host, username, password, realm)
   def apply(host: String, username: String, password: String, realm: String): DirectCredentials =
     DirectCredentials(host, username, password, Option(realm))
-  def apply(host: String, username: String, password: String, realm: Option[String], optional: Boolean): DirectCredentials =
+  def apply(
+    host: String,
+    username: String,
+    password: String,
+    realm: Option[String],
+    optional: Boolean
+  ): DirectCredentials =
     DirectCredentials(host, username, password, realm, optional, matchHost = false, httpsOnly = true)
   def apply(host: String, username: String, password: String, realm: String, optional: Boolean): DirectCredentials =
     DirectCredentials(host, username, password, Option(realm), optional, matchHost = false, httpsOnly = true)

modules/lm-coursier/src/main/scala/lmcoursier/definitions/CachePolicy.scala

@@ -69,4 +69,4 @@ object CachePolicy {
    * Erases files already in cache.
    */
   case object ForceDownload extends CachePolicy
-}
+}

modules/lm-coursier/src/main/scala/lmcoursier/definitions/FromCoursier.scala

@@ -15,4 +15,4 @@ object FromCoursier {
       case coursier.cache.CachePolicy.FetchMissing => CachePolicy.FetchMissing
       case coursier.cache.CachePolicy.ForceDownload => CachePolicy.ForceDownload
     }
-}
+}

modules/lm-coursier/src/main/scala/lmcoursier/definitions/ToCoursier.scala

@@ -61,12 +61,12 @@ object ToCoursier {
       case Reconciliation.SemVer => coursier.core.Reconciliation.SemVer
     }
 
-  def reconciliation(rs: Vector[(ModuleMatchers, Reconciliation)]):
-    Vector[(coursier.util.ModuleMatchers, coursier.core.Reconciliation)] =
+  def reconciliation(rs: Vector[(ModuleMatchers, Reconciliation)])
+    : Vector[(coursier.util.ModuleMatchers, coursier.core.Reconciliation)] =
     rs map { case (m, r) => (moduleMatchers(m), reconciliation(r)) }
 
-  def sameVersions(sv: Seq[Set[InclExclRule]]):
-    Seq[(coursier.params.rule.SameVersion, coursier.params.rule.RuleResolution)] =
+  def sameVersions(sv: Seq[Set[InclExclRule]])
+    : Seq[(coursier.params.rule.SameVersion, coursier.params.rule.RuleResolution)] =
     sv.map { libs =>
       val matchers = libs.map(rule => coursier.util.ModuleMatcher(module(rule.organization, rule.name)))
       coursier.params.rule.SameVersion(matchers) -> coursier.params.rule.RuleResolution.TryResolve

modules/lm-coursier/src/main/scala/lmcoursier/internal/ArtifactsRun.scala

@@ -40,7 +40,7 @@ object ArtifactsRun {
       )
     }
 
-    Lock.maybeSynchronized(needsLock = params.loggerOpt.nonEmpty || !RefreshLogger.defaultFallbackMode){
+    Lock.maybeSynchronized(needsLock = params.loggerOpt.nonEmpty || !RefreshLogger.defaultFallbackMode) {
       result(params, coursierLogger)
     }
   }

modules/lm-coursier/src/main/scala/lmcoursier/internal/InterProjectRepository.scala

@@ -32,4 +32,4 @@ final case class InterProjectRepository(projects: Seq[Project]) extends Reposito
     overrideClassifiers: Option[Seq[Classifier]]
   ) =
     Nil
-}
+}

modules/lm-coursier/src/main/scala/lmcoursier/internal/ResolutionRun.scala

@@ -138,8 +138,7 @@ object ResolutionRun {
                 if (attempt + 1 >= maxAttempts) {
                   log.error(s"Failed, maximum iterations ($maxAttempts) reached")
                   Task.point(Left(e))
-                }
-                else {
+                } else {
                   log.warn(s"Attempt ${attempt + 1} failed: $e")
                   Task.completeAfter(retryScheduler, waitOnError).flatMap { _ =>
                     retry(attempt + 1, waitOnError * 2)
@@ -151,8 +150,7 @@ object ResolutionRun {
               if (attempt + 1 >= maxAttempts) {
                 log.error(s"Failed, maximum iterations ($maxAttempts) reached")
                 Task.fail(e)
-              }
-              else
+              } else
                 Task.completeAfter(retryScheduler, waitOnError).flatMap { _ =>
                   retry(attempt + 1, waitOnError * 2)
                 }
@@ -218,7 +216,7 @@ object ResolutionRun {
   private[internal] def isTransientResolutionError(e: ResolutionError): Boolean =
     e.errors.exists {
       case err: CantDownloadModule => isTimeout(err) || isServerError(err)
-      case _                       => false
+      case _ => false
     }
 
   private def isTimeout(err: CantDownloadModule): Boolean =

modules/lm-coursier/src/main/scala/lmcoursier/internal/Resolvers.scala

@@ -17,8 +17,8 @@ object Resolvers {
 
   private def mavenCompatibleBaseOpt(patterns: Patterns): Option[String] =
     if (patterns.isMavenCompatible) {
-      //input  : /Users/user/custom/repo/[organisation]/[module](_[scalaVersion])(_[sbtVersion])/[revision]/[artifact]-[revision](-[classifier]).[ext]
-      //output : /Users/user/custom/repo/
+      // input  : /Users/user/custom/repo/[organisation]/[module](_[scalaVersion])(_[sbtVersion])/[revision]/[artifact]-[revision](-[classifier]).[ext]
+      // output : /Users/user/custom/repo/
       def basePattern(pattern: String): String = pattern.takeWhile(c => c != '[' && c != '(')
 
       val baseIvyPattern = basePattern(patterns.ivyPatterns.head)
@@ -50,9 +50,9 @@ object Resolvers {
       case e: MalformedURLException =>
         log.warn(
           "Error parsing Maven repository base " +
-          root +
-          Option(e.getMessage).fold("")(" (" + _ + ")") +
-          ", ignoring it"
+            root +
+            Option(e.getMessage).fold("")(" (" + _ + ")") +
+            ", ignoring it"
         )
 
         None
@@ -82,8 +82,8 @@ object Resolvers {
         mavenRepositoryOpt(r.root, log, authentication, classLoaders)
 
       case r: FileRepository
-        if r.patterns.ivyPatterns.lengthCompare(1) == 0 &&
-          r.patterns.artifactPatterns.lengthCompare(1) == 0 =>
+          if r.patterns.ivyPatterns.lengthCompare(1) == 0 &&
+            r.patterns.artifactPatterns.lengthCompare(1) == 0 =>
 
         val mavenCompatibleBaseOpt0 = mavenCompatibleBaseOpt(r.patterns)
 

modules/lm-coursier/src/main/scala/lmcoursier/internal/SbtBootJars.scala

@@ -20,4 +20,4 @@ object SbtBootJars {
           (mod, scalaVersion) -> jar
       }
       .toMap
-}
+}

modules/lm-coursier/src/main/scala/lmcoursier/internal/SbtCoursierCache.scala

@@ -17,7 +17,6 @@ class SbtCoursierCache {
   // are cached
   private val reportsCache = new ConcurrentHashMap[ReportKey, UpdateReport]
 
-
   def resolutionOpt(key: ResolutionKey): Option[Map[Configuration, Resolution]] =
     Option(resolutionsCache.get(key))
   def putResolution(key: ResolutionKey, res: Map[Configuration, Resolution]): Unit =
@@ -59,7 +58,6 @@ object SbtCoursierCache {
     includeSignatures: Boolean
   )
 
-
   // private[coursier]
   val default = new SbtCoursierCache