support additional message

Author: gmethvin

README.md

@@ -23,7 +23,7 @@ scalafix dependency:[email protected]::scalafix-rules:0.1.0
 
 ### NoFutureTraverse
 
-Warns against or disallows the use of `Future.traverse` in your Scala code, to prevent a potentially unbounded number of concurrent tasks from being run at once.
+Warns against or disallows the use of `Future.traverse` and `Future.sequence` in your Scala code, to prevent a potentially unbounded number of concurrent tasks from being run at once.
 
 ```hocon
 rules = [
@@ -32,6 +32,7 @@ rules = [
 ]
 
 NoFutureTraverse {
-  isError = true # Whether to treat violations as errors (default: false)
+  isError = false # Whether to treat violations as errors (default: false)
+  extraMessage = null # Additional deprecation message. Can be used to suggest an alternative.
 }
 ```

rules/src/main/scala/fix/NoFutureTraverse.scala

@@ -8,7 +8,8 @@ import scalafix.lint._
 import scalafix.v1._
 
 case class NoFutureTraverseConfig(
-    isError: Boolean = false
+    isError: Boolean = false,
+    extraMessage: Option[String] = None
 )
 object NoFutureTraverseConfig {
   implicit val surface: Surface[NoFutureTraverseConfig] =
@@ -20,7 +21,7 @@ object NoFutureTraverseConfig {
 class NoFutureTraverse(config: NoFutureTraverseConfig) extends SemanticRule("NoFutureTraverse") {
 
   case class Deprecation(position: Position, name: String) extends Diagnostic {
-    override def message = s"$name is deprecated"
+    override def message = s"$name is deprecated${config.extraMessage.map(s => s": $s").getOrElse("")}"
     override def severity =
       if (config.isError) LintSeverity.Error else LintSeverity.Warning
   }