introduce code formatting

.github/workflows/ci.yml

@@ -24,6 +24,9 @@ jobs:
           distribution: 'oracle'
           cache: 'sbt'
 
+      - name: Check code formatting
+        run: sbt check
+
       - name: build ${{ matrix.scala }}
         run: sbt ++${{ matrix.scala }} clean coverage test
 

.scalafix.conf

@@ -0,0 +1,6 @@
+rules = [OrganizeImports]
+
+OrganizeImports {
+  preset = INTELLIJ_2020_3
+  targetDialect = Auto
+}

.scalafmt.conf

@@ -0,0 +1,19 @@
+version = 3.8.1
+
+maxColumn = 120
+
+preset = default
+align.preset = none
+
+trailingCommas = always
+
+continuationIndent {
+  callSite = 2
+  defnSite = 2
+}
+
+rewrite.rules = [
+  RedundantBraces, RedundantParens, SortModifiers, prefercurlyfors
+]
+
+runner.dialect = Scala213Source3

build.sbt

@@ -8,13 +8,21 @@ organizationHomepage := Some(url("https://evolution.com"))
 homepage := Some(url("https://github.com/evolution-gaming/resource-pool"))
 startYear := Some(2023)
 
-crossScalaVersions := Seq("2.13.14")
-scalaVersion := crossScalaVersions.value.head
-scalacOptions := Seq(
-  "-release:17",
-  "-Xsource:3",
-  "-deprecation",
+inThisBuild(
+  List(
+    crossScalaVersions := Seq("2.13.14"),
+    scalaVersion := crossScalaVersions.value.head,
+    scalacOptions := Seq(
+      "-release:17",
+      "-Xsource:3",
+      "-deprecation",
+      "-Wunused:imports",
+    ),
+    semanticdbEnabled := true,
+    semanticdbVersion := scalafixSemanticdb.revision,
+  ),
 )
+
 releaseCrossBuild := true
 autoAPIMappings := true
 versionScheme := Some("early-semver")
@@ -29,3 +37,7 @@ libraryDependencies ++= Seq(
 licenses := Seq(("MIT", url("https://opensource.org/licenses/MIT")))
 
 addCommandAlias("build", "all compile test")
+
+// https://github.com/scalacenter/scalafix/issues/1488
+addCommandAlias("check", "scalafixAll --check; all scalafmtCheckAll scalafmtSbtCheck")
+addCommandAlias("fix", "scalafixAll; all scalafmtAll scalafmtSbt")

project/Dependencies.scala

@@ -1,7 +1,7 @@
 import sbt._
 
 object Dependencies {
-  val scalatest        = "org.scalatest" %% "scalatest"      % "3.2.18" % Test
-  val `kind-projector` = "org.typelevel"  % "kind-projector" % "0.13.3"
-  val `cats-effect`    = "org.typelevel" %% "cats-effect"    % "3.5.4"
+  val scalatest = "org.scalatest" %% "scalatest" % "3.2.18" % Test
+  val `kind-projector` = "org.typelevel" % "kind-projector" % "0.13.3"
+  val `cats-effect` = "org.typelevel" %% "cats-effect" % "3.5.4"
 }

project/plugins.sbt

@@ -6,4 +6,8 @@ addSbtPlugin("com.github.sbt" % "sbt-release" % "1.4.0")
 
 addSbtPlugin("com.evolution" % "sbt-scalac-opts-plugin" % "0.0.9")
 
-addSbtPlugin("com.evolution" % "sbt-artifactory-plugin" % "0.0.2")
+addSbtPlugin("com.evolution" % "sbt-artifactory-plugin" % "0.0.2")
+
+addSbtPlugin("ch.epfl.scala" % "sbt-scalafix" % "0.12.1")
+
+addSbtPlugin("org.scalameta" % "sbt-scalafmt" % "2.5.2")

src/main/scala/com/evolution/resourcepool/IntHelper.scala

@@ -6,29 +6,25 @@ private[resourcepool] object IntHelper {
 
   implicit class IntOpsIntHelper(val self: Int) extends AnyVal {
 
-    def divide(value: Int): List[Int] = {
+    def divide(value: Int): List[Int] =
       if (value <= 0 || self <= 0) {
         List.empty
       } else if (value >= self) {
         List.fill(self)(1)
       } else if (value == 1) {
         List(self)
       } else {
-        val quotient = (self.toDouble / value)
-          .round
-          .toInt
+        val quotient = (self.toDouble / value).round.toInt
 
         @tailrec
-        def loop(self: Int, value: Int, result: List[Int]): List[Int] = {
+        def loop(self: Int, value: Int, result: List[Int]): List[Int] =
           if (self > quotient && value > 1) {
             loop(self - quotient, value - 1, quotient :: result)
           } else {
             (self :: result).reverse
           }
-        }
 
         loop(self, value, List.empty)
       }
-    }
   }
 }