Merge branch 'master' into update/scalatest-3.2.19

Author: muuki88

.git-blame-ignore-revs

@@ -0,0 +1,2 @@
+# Scala Steward: Reformat with scalafmt 3.8.3
+fb3b1e158c70f5b76207764dc42b128cb878b3be

.scalafmt.conf

@@ -1,5 +1,5 @@
 # Version https://scalameta.org/scalafmt/docs/configuration.html#version
-version = 3.8.1
+version = 3.8.3
 # Dialect https://scalameta.org/scalafmt/docs/configuration.html#scala-dialects
 runner.dialect = scala212
 

integration-tests-ansible/test-project-play-rpm/build.sbt

@@ -1,14 +1,6 @@
 scalaVersion in ThisBuild := "2.11.6"
 
-scalacOptions in ThisBuild ++= Seq(
-  "-deprecation",
-  "-encoding",
-  "UTF-8",
-  "-feature",
-  "-unchecked",
-  "-Xfuture",
-  "-Xlint"
-)
+scalacOptions in ThisBuild ++= Seq("-deprecation", "-encoding", "UTF-8", "-feature", "-unchecked", "-Xfuture", "-Xlint")
 
 name := "test-project-play-rpm"
 

src/main/scala-sbt-0.13/com/typesafe/sbt/packager/Compat.scala

@@ -9,14 +9,15 @@ object Compat {
   /**
     * Used in:
     *
-    * - [[com.typesafe.sbt.packager.windows.WindowsPlugin]]
-    * - [[com.typesafe.sbt.packager.rpm.RpmHelper]]
-    * - [[com.typesafe.sbt.packager.docker.DockerPlugin]]
-    * - [[com.typesafe.sbt.packager.debian.DebianNativePackaging]]
-    * - [[com.typesafe.sbt.packager.rpm.RpmPlugin]]
+    *   - [[com.typesafe.sbt.packager.windows.WindowsPlugin]]
+    *   - [[com.typesafe.sbt.packager.rpm.RpmHelper]]
+    *   - [[com.typesafe.sbt.packager.docker.DockerPlugin]]
+    *   - [[com.typesafe.sbt.packager.debian.DebianNativePackaging]]
+    *   - [[com.typesafe.sbt.packager.rpm.RpmPlugin]]
     *
     * @param log
-    * @return turns a Logger into a ProcessLogger
+    * @return
+    *   turns a Logger into a ProcessLogger
     */
   implicit def log2ProcessLogger(log: Logger): sys.process.ProcessLogger =
     new BufferedLogger(new FullLogger(log)) with sys.process.ProcessLogger {
@@ -27,17 +28,20 @@ object Compat {
   /**
     * Used in
     *
-    * - [[com.typesafe.sbt.packager.docker.DockerPlugin]]
+    *   - [[com.typesafe.sbt.packager.docker.DockerPlugin]]
     *
-    * @param logger The sbt.ProcessLogger that should be wrapped
-    * @return A scala ProcessLogger
+    * @param logger
+    *   The sbt.ProcessLogger that should be wrapped
+    * @return
+    *   A scala ProcessLogger
     */
   implicit def sbtProcessLogger2ScalaProcessLogger(logger: sbt.ProcessLogger): sys.process.ProcessLogger =
     ProcessLogger(msg => logger.info(msg), err => logger.error(err))
 
   /**
     * Use in the scripted `universal/multiproject-classifiers` test.
-    * @param artifact polyfill new methods
+    * @param artifact
+    *   polyfill new methods
     */
   implicit class CompatArtifact(artifact: Artifact) {
     def withClassifier(classifier: Option[String]): Artifact =

src/main/scala-sbt-0.13/com/typesafe/sbt/packager/MappingsHelper.scala

@@ -9,12 +9,13 @@ object MappingsHelper {
     * return a Seq of mappings which effect is to add a whole directory in the generated package
     *
     * @example
-    * {{{
+    *   {{{
     * mappings in Universal ++= directory(baseDirectory.value / "extra")
-    * }}}
+    *   }}}
     *
     * @param sourceDir
-    * @return mappings
+    * @return
+    *   mappings
     */
   def directory(sourceDir: File): Seq[(File, String)] =
     Option(sourceDir.getParentFile)
@@ -25,27 +26,29 @@ object MappingsHelper {
     * It lightens the build file if one wants to give a string instead of file.
     *
     * @example
-    * {{{
+    *   {{{
     * mappings in Universal ++= directory("extra")
-    * }}}
+    *   }}}
     *
     * @param sourceDir
-    * @return mappings
+    * @return
+    *   mappings
     */
   def directory(sourceDir: String): Seq[(File, String)] =
     directory(file(sourceDir))
 
   /**
-    * return a Seq of mappings which effect is to add the content of directory in the generated package,
-    * excluding the directory itself.
+    * return a Seq of mappings which effect is to add the content of directory in the generated package, excluding the
+    * directory itself.
     *
     * @example
-    * {{{
+    *   {{{
     * mappings in Universal ++= sourceDir(baseDirectory.value / "extra")
-    * }}}
+    *   }}}
     *
     * @param sourceDir
-    * @return mappings
+    * @return
+    *   mappings
     */
   def contentOf(sourceDir: File): Seq[(File, String)] =
     (sourceDir.*** --- sourceDir) pair relativeTo(sourceDir)
@@ -54,50 +57,57 @@ object MappingsHelper {
     * It lightens the build file if one wants to give a string instead of file.
     *
     * @example
-    * {{{
+    *   {{{
     * mappings in Universal ++= sourceDir("extra")
-    * }}}
+    *   }}}
     *
-    * @param sourceDir as string representation
-    * @return mappings
+    * @param sourceDir
+    *   as string representation
+    * @return
+    *   mappings
     */
   def contentOf(sourceDir: String): Seq[(File, String)] =
     contentOf(file(sourceDir))
 
   /**
-    * Create mappings from your classpath. For example if you want to add additional
-    * dependencies, like test or model.
+    * Create mappings from your classpath. For example if you want to add additional dependencies, like test or model.
     *
-    * @example Add all test artifacts to a separated test folder
-    * {{{
+    * @example
+    *   Add all test artifacts to a separated test folder
+    *   {{{
     * mappings in Universal ++= fromClasspath((managedClasspath in Test).value, target = "test")
-    * }}}
+    *   }}}
     *
     * @param entries
     * @param target
-    * @return a list of mappings
+    * @return
+    *   a list of mappings
     */
   def fromClasspath(entries: Seq[Attributed[File]], target: String): Seq[(File, String)] =
     fromClasspath(entries, target, _ => true)
 
   /**
-    * Create mappings from your classpath. For example if you want to add additional
-    * dependencies, like test or model. You can also filter the artifacts that should
-    * be mapped to mappings.
+    * Create mappings from your classpath. For example if you want to add additional dependencies, like test or model.
+    * You can also filter the artifacts that should be mapped to mappings.
     *
-    * @example Filter all osgi bundles
-    * {{{
+    * @example
+    *   Filter all osgi bundles
+    *   {{{
     * mappings in Universal ++= fromClasspath(
     *    (managedClasspath in Runtime).value,
     *    "osgi",
     *    artifact => artifact.`type` == "bundle"
     * )
-    * }}}
+    *   }}}
     *
-    * @param entries from where mappings should be created from
-    * @param target folder, e.g. `model`. Must not end with a slash
-    * @param includeArtifact function to determine if an artifact should result in a mapping
-    * @param includeOnNoArtifact default is false. When there's no Artifact meta data remove it
+    * @param entries
+    *   from where mappings should be created from
+    * @param target
+    *   folder, e.g. `model`. Must not end with a slash
+    * @param includeArtifact
+    *   function to determine if an artifact should result in a mapping
+    * @param includeOnNoArtifact
+    *   default is false. When there's no Artifact meta data remove it
     */
   def fromClasspath(
     entries: Seq[Attributed[File]],

src/main/scala-sbt-0.13/com/typesafe/sbt/packager/SettingsHelper.scala

@@ -72,8 +72,8 @@ object SettingsHelper {
     ) ++ addPackage(config, packageTask, extension, classifier) ++ addResolver(config)
 
   /**
-    * SBT looks in the `otherResolvers` setting for resolvers defined in `publishTo`.
-    * If a user scopes a `publishTo`, e.g.
+    * SBT looks in the `otherResolvers` setting for resolvers defined in `publishTo`. If a user scopes a `publishTo`,
+    * e.g.
     *
     * {{{
     * // publish the rpm to the target folder
@@ -82,7 +82,8 @@ object SettingsHelper {
     *
     * then the resolver must also be present in the `otherResolvers`
     *
-    * @param config the ivy configuration to look for resolvers
+    * @param config
+    *   the ivy configuration to look for resolvers
     */
   private def addResolver(config: Configuration): Seq[Setting[_]] =
     Seq(otherResolvers ++= (publishTo in config).value.toSeq)

src/main/scala/com/typesafe/sbt/packager/docker/DockerPlugin.scala

@@ -383,7 +383,7 @@ object DockerPlugin extends AutoPlugin {
     *   FROM command
     */
   private final def makeFromAs(dockerBaseImage: String, name: String): CmdLike =
-    Cmd("FROM", dockerBaseImage, "as", name)
+    Cmd("FROM", dockerBaseImage, "AS", name)
 
   /**
     * @param label

src/sbt-test/ash/add-app-settings/build.sbt

@@ -15,9 +15,9 @@ TaskKey[Unit]("scriptCheck") := {
 
 TaskKey[Unit]("runCheck") := {
   val cwd = (stagingDirectory in Universal).value
-  val cmd = Seq((cwd / "bin" / packageName.value).getAbsolutePath)  
+  val cmd = Seq((cwd / "bin" / packageName.value).getAbsolutePath)
   val output = (sys.process.Process(cmd, cwd).!!).replaceAll("\n", "")
-  
+
   assert(output.contains("info"), s"Application did not receive residual arg 'info'")
   assert(output.contains("help"), s"Application did not receive residual arg 'help'")
 }

src/sbt-test/ash/add-app-settings/src/main/scala/MainApp.scala

@@ -1,3 +1,3 @@
-object MainApp extends App {  
+object MainApp extends App {
   println(args.mkString("|"))
 }

src/sbt-test/ash/command-line-settings/build.sbt

@@ -23,7 +23,23 @@ TaskKey[Unit]("checkResidual") := {
 }
 
 TaskKey[Unit]("checkComplexResidual") := {
-  val args = Seq("-J-Dfoo=bar", "arg1", "--", "-J-Dfoo=bar", "arg 2", "--", "\"", "$foo", "'", "%s", "-y", "bla", "\\'", "\\\"", "''")
+  val args = Seq(
+    "-J-Dfoo=bar",
+    "arg1",
+    "--",
+    "-J-Dfoo=bar",
+    "arg 2",
+    "--",
+    "\"",
+    "$foo",
+    "'",
+    "%s",
+    "-y",
+    "bla",
+    "\\'",
+    "\\\"",
+    "''"
+  )
   val cwd = (stagingDirectory in Universal).value
   val cmd = Seq((cwd / "bin" / packageName.value).getAbsolutePath) ++ args
   val expected = """arg1|-J-Dfoo=bar|arg 2|--|"|$foo|'|%s|-y|bla|\'|\"|''"""