sttp icon indicating copy to clipboard operation
sttp copied to clipboard

Spray json support for scala 3

Open MichelEdkrantz opened this issue 9 months ago • 1 comments

Hi, reaching out since my company is using spray json for a lot of things, and we also like your libraries :)

Wanted to check regarding spray json, and scala 3 support. Spray json also has a Scala 3 build, but not linked in your sbt build. Any historic reason for this or not doing this? It might be as easy as adding scala3 below.

If I opened a PR for this, should it be for the client3 or client4?

lazy val sprayJson = (projectMatrix in file("json/spray-json"))
  .settings(commonJvmSettings)
  .settings(
    name := "spray-json",
    libraryDependencies ++= Seq(
      "io.spray" %% "spray-json" % "1.3.6"
    ),
    scalaTest
  )
  .jvmPlatform(scalaVersions = scala2)
  .dependsOn(core, jsonCommon)

MichelEdkrantz avatar May 13 '24 06:05 MichelEdkrantz

Probably nobody submitted a patch yet - no other reasons, if the tests pass. Can you maybe create a PR?

adamw avatar May 13 '24 07:05 adamw

If I opened a PR for this, should it be for the client3 or client4?

I think it's worth adding for client3 and client4.

kciesielski avatar May 15 '24 07:05 kciesielski

Added another PR here for v3 - #2186

MichelEdkrantz avatar May 20 '24 10:05 MichelEdkrantz