Compile without fatal warnings on 2.13.0

core/src/main/scala/org/http4s/Message.scala

@@ -6,12 +6,13 @@ import cats.implicits._
 import cats.effect.IO
 import fs2.{Pure, Stream}
 import fs2.text.{utf8Decode, utf8Encode}
-import java.io.File
+import java.io.{File, Serializable}
 import java.net.{InetAddress, InetSocketAddress}
 import org.http4s.headers._
 import org.http4s.util.decode
 import org.log4s.getLogger
 import _root_.io.chrisdavenport.vault._
+import scala.util.hashing.MurmurHash3
 
 /**
   * Represents a HTTP Message. The interesting subclasses are Request and Response.
@@ -236,14 +237,14 @@ object Message {
   * @param body fs2.Stream[F, Byte] defining the body of the request
   * @param attributes Immutable Map used for carrying additional information in a type safe fashion
   */
-sealed abstract case class Request[F[_]](
-    method: Method = Method.GET,
-    uri: Uri = Uri(path = "/"),
-    httpVersion: HttpVersion = HttpVersion.`HTTP/1.1`,
-    headers: Headers = Headers.empty,
-    body: EntityBody[F] = EmptyBody,
-    attributes: Vault = Vault.empty
-) extends Message[F] {
+final class Request[F[_]](
+    val method: Method = Method.GET,
+    val uri: Uri = Uri(path = "/"),
+    val httpVersion: HttpVersion = HttpVersion.`HTTP/1.1`,
+    val headers: Headers = Headers.empty,
+    val body: EntityBody[F] = EmptyBody,
+    val attributes: Vault = Vault.empty
+) extends Message[F] with Product with Serializable {
   import Request._
 
   type Self = Request[F]
@@ -414,6 +415,34 @@ sealed abstract case class Request[F[_]](
   override def toString: String =
     s"""Request(method=$method, uri=$uri, headers=${headers.redactSensitive()})"""
 
+  override def hashCode(): Int = MurmurHash3.productHash(this)
+
+  override def equals(that: Any): Boolean = (this eq that.asInstanceOf[Object]) || (that match {
+    case that: Request[_] =>
+      (this.method == that.method) &&
+      (this.uri == that.uri) &&
+      (this.httpVersion == that.httpVersion) &&
+      (this.headers == that.headers) &&
+      (this.body == that.body) &&
+      (this.attributes == that.attributes)
+    case _ => false
+  })
+
+  def canEqual(that: Any) = that.isInstanceOf[Request[F]]
+
+  def productArity: Int = 6
+
+  def productElement(n: Int) = n match {
+    case 0 => method
+    case 1 => uri
+    case 2 => httpVersion
+    case 3 => headers
+    case 4 => body
+    case 5 => attributes
+    case _ => throw new IndexOutOfBoundsException()
+  }
+
+  override def productPrefix: String = "Request"
 }
 
 object Request {
@@ -433,7 +462,10 @@ object Request {
       headers = headers,
       body = body,
       attributes = attributes
-    ) {}
+    )
+
+  def unapply[F[_]](request: Request[F]): Option[(Method, Uri, HttpVersion, Headers, EntityBody[F], Vault)] =
+    Some((request.method, request.uri, request.httpVersion, request.headers, request.body, request.attributes))
 
   final case class Connection(local: InetSocketAddress, remote: InetSocketAddress, secure: Boolean)
 

dsl/src/test/scala/org/http4s/dsl/PathInHttpRoutesSpec.scala

@@ -10,8 +10,8 @@ import org.http4s.Uri.uri
 object PathInHttpRoutesSpec extends Http4sSpec {
 
   object List {
-    def unapplySeq(params: Map[String, Seq[String]]) = params.get("list")
-    def unapply(params: Map[String, Seq[String]]) = unapplySeq(params)
+    def unapplySeq(params: Map[String, collection.Seq[String]]) = params.get("list")
+    def unapply(params: Map[String, collection.Seq[String]]) = unapplySeq(params)
   }
 
   object I extends QueryParamDecoderMatcher[Int]("start")

project/Http4sPlugin.scala

@@ -57,15 +57,6 @@ object Http4sPlugin extends AutoPlugin {
   override lazy val projectSettings: Seq[Setting[_]] = Seq(
     scalaVersion := scala_213,
     crossScalaVersions := Seq(scala_213, scala_212),
-    // Getting some spurious unreachable code warnings in 2.13.0
-    scalacOptions -= {
-      CrossVersion.partialVersion(scalaVersion.value) match {
-        case Some((2, 13)) =>
-          "-Xfatal-warnings"
-        case _ =>
-          "I DON'T EXIST I'M WORKING AROUND NOT BEING ABLE TO CALL scalaVersion.value FROM ~="
-      }
-    },
 
     // https://github.com/tkawachi/sbt-doctest/issues/102
     Test / compile / scalacOptions -= "-Ywarn-unused:params",

tomcat/src/test/scala/org/http4s/tomcat/TomcatServerSpec.scala

@@ -13,14 +13,11 @@ import org.specs2.concurrent.ExecutionEnv
 import scala.concurrent.duration._
 import scala.io.Source
 
-class TomcatServerSpec(implicit ee: ExecutionEnv) extends {
+class TomcatServerSpec(implicit ee: ExecutionEnv) extends Http4sSpec {
   // Prevents us from loading jar and war URLs, but lets us
   // run Tomcat twice in the same JVM.  This makes me grumpy.
-  //
-  // Needs to run before the server is initialized in the superclass.
-  // This also makes me grumpy.
-  val _ = TomcatURLStreamHandlerFactory.disable()
-} with Http4sSpec {
+  TomcatURLStreamHandlerFactory.disable()
+
   def builder = TomcatBuilder[IO]
 
   val serverR =