Another scalafmt (#382)

scapegoat-scala · Jun 17, 2020 · 5750a81 · 5750a81
1 parent f42e9ea
commit 5750a81
Show file tree

Hide file tree

Showing 105 changed files with 207 additions and 106 deletions.
diff --git a/src/main/scala/com/sksamuel/scapegoat/Feedback.scala b/src/main/scala/com/sksamuel/scapegoat/Feedback.scala
@@ -4,7 +4,8 @@ import scala.collection.mutable.ListBuffer
 import scala.reflect.internal.util.Position
 import scala.tools.nsc.reporters.Reporter
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class Feedback(
   consoleOutput: Boolean,
   reporter: Reporter,

diff --git a/src/main/scala/com/sksamuel/scapegoat/Inspection.scala b/src/main/scala/com/sksamuel/scapegoat/Inspection.scala
@@ -3,7 +3,8 @@ package com.sksamuel.scapegoat
 import scala.reflect.internal.util.Position
 import scala.tools.nsc.Global
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 abstract class Inspection(
   val text: String,
   val defaultLevel: Level,

diff --git a/src/main/scala/com/sksamuel/scapegoat/Level.scala b/src/main/scala/com/sksamuel/scapegoat/Level.scala
@@ -1,6 +1,7 @@
 package com.sksamuel.scapegoat
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 sealed trait Level
 
 object Levels {

diff --git a/src/main/scala/com/sksamuel/scapegoat/ScapegoatConfig.scala b/src/main/scala/com/sksamuel/scapegoat/ScapegoatConfig.scala
@@ -18,7 +18,8 @@ import com.sksamuel.scapegoat.inspections.style._
 import com.sksamuel.scapegoat.inspections.unneccesary._
 import com.sksamuel.scapegoat.inspections.unsafe._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 object ScapegoatConfig extends App {
 
   def inspections: Seq[Inspection] =

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/AnyUse.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/AnyUse.scala
@@ -4,7 +4,8 @@ import scala.reflect.internal.Flags
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class AnyUse
     extends Inspection(
       text = "Use of Any",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/AvoidToMinusOne.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/AvoidToMinusOne.scala
@@ -4,7 +4,8 @@ import scala.runtime.{RichInt, RichLong}
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class AvoidToMinusOne
     extends Inspection(
       text = "Avoid (j to k - 1)",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/DoubleNegation.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/DoubleNegation.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class DoubleNegation
     extends Inspection(
       text = "Double negation",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/EmptyCaseClass.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/EmptyCaseClass.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class EmptyCaseClass
     extends Inspection(
       text = "Empty case class",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/LonelySealedTrait.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/LonelySealedTrait.scala
@@ -4,7 +4,8 @@ import scala.collection.mutable
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class LonelySealedTrait
     extends Inspection(
       text = "Lonely sealed trait",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/MaxParameters.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/MaxParameters.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class MaxParameters
     extends Inspection(
       text = "Max parameters",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/NoOpOverride.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/NoOpOverride.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class NoOpOverride
     extends Inspection(
       text = "Noop override",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/PublicFinalizer.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/PublicFinalizer.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class PublicFinalizer
     extends Inspection(
       text = "PublicFinalizer",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/TypeShadowing.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/TypeShadowing.scala
@@ -4,7 +4,8 @@ import scala.collection.mutable
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class TypeShadowing
     extends Inspection(
       text = "Type shadowing",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/VarClosure.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/VarClosure.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class VarClosure
     extends Inspection(
       text = "Var in closure",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ArrayEquals.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ArrayEquals.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class ArrayEquals
     extends Inspection(
       text = "Array equals",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/AvoidSizeEqualsZero.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/AvoidSizeEqualsZero.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class AvoidSizeEqualsZero
     extends Inspection(
       text = "Avoid Traversable.size == 0",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/AvoidSizeNotEqualsZero.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/AvoidSizeNotEqualsZero.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class AvoidSizeNotEqualsZero
     extends Inspection(
       text = "Avoid Traversable.size != 0",

diff --git a/...scala/com/sksamuel/scapegoat/inspections/collections/CollectionIndexOnNonIndexedSeq.scala b/...scala/com/sksamuel/scapegoat/inspections/collections/CollectionIndexOnNonIndexedSeq.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Josh Rosen */
+/**
+ * @author Josh Rosen */
 class CollectionIndexOnNonIndexedSeq
     extends Inspection(
       text = "Use of apply method on a non-indexed Seq",

diff --git a/...main/scala/com/sksamuel/scapegoat/inspections/collections/CollectionNamingConfusion.scala b/...main/scala/com/sksamuel/scapegoat/inspections/collections/CollectionNamingConfusion.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class CollectionNamingConfusion
     extends Inspection(
       text = "Collection naming confusion",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/CollectionNegativeIndex.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/CollectionNegativeIndex.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class CollectionNegativeIndex
     extends Inspection(
       text = "Collection index out of bounds",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ComparisonToEmptyList.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ComparisonToEmptyList.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class ComparisonToEmptyList
     extends Inspection(
       text = "Comparison to empty list",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ComparisonToEmptySet.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ComparisonToEmptySet.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class ComparisonToEmptySet
     extends Inspection(
       text = "Comparison to empty set",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/DuplicateMapKey.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/DuplicateMapKey.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class DuplicateMapKey
     extends Inspection(
       text = "Duplicated map key",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/DuplicateSetValue.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/DuplicateSetValue.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class DuplicateSetValue
     extends Inspection(
       text = "Duplicated set value",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterDotHead.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterDotHead.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class FilterDotHead
     extends Inspection(
       text = "filter().head can throw an exception",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterDotHeadOption.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterDotHeadOption.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class FilterDotHeadOption
     extends Inspection(
       text = "filter().headOption instead of find()",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterDotIsEmpty.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterDotIsEmpty.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class FilterDotIsEmpty
     extends Inspection(
       text = "filter().isEmpty instead of !exists()",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterOptionAndGet.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FilterOptionAndGet.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class FilterOptionAndGet
     extends Inspection(
       text = "filter(_.isDefined).map(_.get) instead of flatten",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FindDotIsDefined.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/FindDotIsDefined.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class FindDotIsDefined
     extends Inspection(
       text = "find().isDefined() instead of exists()",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/JavaConversionsUse.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/JavaConversionsUse.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class JavaConversionsUse
     extends Inspection(
       text = "Java conversions",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ListAppend.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ListAppend.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class ListAppend
     extends Inspection(
       text = "List append is slow",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ListSize.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/ListSize.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class ListSize
     extends Inspection(
       text = "List.size is O(n)",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/NegationIsEmpty.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/NegationIsEmpty.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class NegationIsEmpty
     extends Inspection(
       text = "!isEmpty can be replaced with nonEmpty",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/NegationNonEmpty.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/NegationNonEmpty.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class NegationNonEmpty
     extends Inspection(
       text = "!nonEmpty can be replaced with isEmpty",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/NegativeSeqPad.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/NegativeSeqPad.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class NegativeSeqPad
     extends Inspection(
       text = "Negative seq padTo",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PredefIterableIsMutable.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PredefIterableIsMutable.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class PredefIterableIsMutable
     extends Inspection(
       text = "Default Iterable is mutable",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PredefSeqIsMutable.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PredefSeqIsMutable.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class PredefSeqIsMutable
     extends Inspection(
       text = "Predef.Seq is mutable",

diff --git a/...ain/scala/com/sksamuel/scapegoat/inspections/collections/PredefTraversableIsMutable.scala b/...ain/scala/com/sksamuel/scapegoat/inspections/collections/PredefTraversableIsMutable.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat.{Inspection, InspectionContext, Inspector, Levels}
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class PredefTraversableIsMutable
     extends Inspection(
       text = "Traversable is mutable",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PreferSeqEmpty.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PreferSeqEmpty.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class PreferSeqEmpty
     extends Inspection(
       text = "Prefer Seq.empty",

diff --git a/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PreferSetEmpty.scala b/src/main/scala/com/sksamuel/scapegoat/inspections/collections/PreferSetEmpty.scala
@@ -2,7 +2,8 @@ package com.sksamuel.scapegoat.inspections.collections
 
 import com.sksamuel.scapegoat._
 
-/** @author Stephen Samuel */
+/**
+ * @author Stephen Samuel */
 class PreferSetEmpty
     extends Inspection(
       text = "Prefer Set.empty",