155 lines
5.6 KiB
Scala
155 lines
5.6 KiB
Scala
/*
|
|
* Copyright (C) 2019-2020 Lightbend Inc. <https://www.lightbend.com>
|
|
*/
|
|
|
|
package akka
|
|
|
|
import sbt._
|
|
import Keys.{ scalacOptions, _ }
|
|
import sbt.plugins.JvmPlugin
|
|
|
|
object AkkaDisciplinePlugin extends AutoPlugin {
|
|
|
|
override def trigger: PluginTrigger = allRequirements
|
|
override def requires: Plugins = JvmPlugin
|
|
override lazy val projectSettings = disciplineSettings
|
|
|
|
// allow toggling for pocs/exploration of ideas without discpline
|
|
val enabled = !sys.props.contains("akka.no.discipline")
|
|
|
|
// We allow warnings in docs to get the 'snippets' right
|
|
val nonFatalJavaWarningsFor = Set(
|
|
// for sun.misc.Unsafe and AbstractScheduler
|
|
"akka-actor",
|
|
// references to deprecated PARSER fields in generated message formats?
|
|
"akka-actor-typed-tests",
|
|
// references to deprecated PARSER fields in generated message formats?
|
|
"akka-cluster-typed",
|
|
// use of deprecated akka.protobuf.GeneratedMessage
|
|
"akka-protobuf",
|
|
"akka-protobuf-v3",
|
|
// references to deprecated PARSER fields in generated message formats?
|
|
"akka-remote",
|
|
// references to deprecated PARSER fields in generated message formats?
|
|
"akka-distributed-data",
|
|
// references to deprecated PARSER fields in generated message formats?
|
|
"akka-cluster-sharding-typed",
|
|
// references to deprecated PARSER fields in generated message formats?
|
|
"akka-persistence-typed",
|
|
"akka-docs")
|
|
|
|
val looseProjects = Set(
|
|
"akka-actor",
|
|
"akka-actor-testkit-typed",
|
|
"akka-actor-tests",
|
|
"akka-actor-typed",
|
|
"akka-actor-typed-tests",
|
|
"akka-bench-jmh",
|
|
"akka-cluster",
|
|
"akka-cluster-metrics",
|
|
"akka-cluster-sharding",
|
|
"akka-cluster-sharding-typed",
|
|
"akka-distributed-data",
|
|
"akka-docs",
|
|
"akka-persistence",
|
|
"akka-persistence-tck",
|
|
"akka-persistence-typed",
|
|
"akka-persistence-query",
|
|
"akka-remote",
|
|
"akka-remote-tests",
|
|
"akka-stream",
|
|
"akka-stream-testkit",
|
|
"akka-stream-tests",
|
|
"akka-stream-tests-tck",
|
|
"akka-testkit")
|
|
|
|
lazy val silencerSettings = {
|
|
val silencerVersion = "1.7.1"
|
|
Seq(
|
|
libraryDependencies ++= Seq(
|
|
compilerPlugin(("com.github.ghik" %% "silencer-plugin" % silencerVersion).cross(CrossVersion.patch)),
|
|
("com.github.ghik" %% "silencer-lib" % silencerVersion % Provided).cross(CrossVersion.patch)))
|
|
}
|
|
|
|
lazy val disciplineSettings =
|
|
if (enabled) {
|
|
silencerSettings ++ Seq(
|
|
Compile / scalacOptions ++= Seq("-Xfatal-warnings"),
|
|
Test / scalacOptions --= testUndicipline,
|
|
Compile / javacOptions ++= (
|
|
if (!nonFatalJavaWarningsFor(name.value)) Seq("-Werror", "-Xlint:deprecation", "-Xlint:unchecked")
|
|
else Seq.empty
|
|
),
|
|
Compile / javacOptions in doc := Seq("-Xdoclint:none"),
|
|
Compile / scalacOptions ++= (CrossVersion.partialVersion(scalaVersion.value) match {
|
|
case Some((2, 13)) =>
|
|
disciplineScalacOptions -- Set(
|
|
"-Ywarn-inaccessible",
|
|
"-Ywarn-infer-any",
|
|
"-Ywarn-nullary-override",
|
|
"-Ywarn-nullary-unit",
|
|
"-Ypartial-unification",
|
|
"-Yno-adapted-args")
|
|
case Some((2, 12)) =>
|
|
disciplineScalacOptions
|
|
case _ =>
|
|
Nil
|
|
}).toSeq,
|
|
Compile / scalacOptions --=
|
|
(if (looseProjects.contains(name.value)) undisciplineScalacOptions.toSeq
|
|
else Seq.empty),
|
|
// Discipline is not needed for the docs compilation run (which uses
|
|
// different compiler phases from the regular run), and in particular
|
|
// '-Ywarn-unused:explicits' breaks 'sbt ++2.13.0-M5 akka-actor/doc'
|
|
// https://github.com/akka/akka/issues/26119
|
|
Compile / doc / scalacOptions --= disciplineScalacOptions.toSeq :+ "-Xfatal-warnings",
|
|
// having discipline warnings in console is just an annoyance
|
|
Compile / console / scalacOptions --= disciplineScalacOptions.toSeq)
|
|
} else {
|
|
// we still need these in opt-out since the annotations are present
|
|
silencerSettings ++ Seq(Compile / scalacOptions += "-deprecation")
|
|
}
|
|
|
|
val testUndicipline = Seq("-Ywarn-dead-code" // '???' used in compile only specs
|
|
)
|
|
|
|
/**
|
|
* Remain visibly filtered for future code quality work and removing.
|
|
*/
|
|
val undisciplineScalacOptions = Set("-Ywarn-numeric-widen")
|
|
|
|
/** These options are desired, but some are excluded for the time being*/
|
|
val disciplineScalacOptions = Set(
|
|
"-Ywarn-numeric-widen",
|
|
"-Yno-adapted-args",
|
|
"-deprecation",
|
|
"-Xlint",
|
|
"-Ywarn-dead-code",
|
|
"-Ywarn-inaccessible",
|
|
"-Ywarn-infer-any",
|
|
"-Ywarn-nullary-override",
|
|
"-Ywarn-nullary-unit",
|
|
"-Ywarn-unused:_",
|
|
"-Ypartial-unification",
|
|
"-Ywarn-extra-implicit")
|
|
|
|
/**
|
|
* We are a little less strict in docs
|
|
*/
|
|
val docs = Seq(
|
|
scalacOptions ++= Seq(
|
|
// In docs, 'unused' variables can be useful for naming and showing the type
|
|
"-P:silencer:globalFilters=is never used",
|
|
// Import statements are often duplicated across multiple snippets in one file
|
|
"-P:silencer:globalFilters=Unused import",
|
|
// We keep documentation for this old API around for a while:
|
|
"-P:silencer:globalFilters=in object Dns is deprecated",
|
|
"-P:silencer:globalFilters=in class Dns is deprecated",
|
|
// Because we sometimes wrap things in a class:
|
|
"-P:silencer:globalFilters=The outer reference in this type test cannot be checked at run time",
|
|
// Because we show some things that are deprecated in
|
|
// 2.13 but don't have a replacement that was in 2.12:
|
|
"-P:silencer:globalFilters=deprecated \\(since 2.13.0\\)"
|
|
)
|
|
)
|
|
}
|