-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates - Scala 2.13 & Akka 2.6.0-M3 (bp #354) #356
Conversation
c13b565
to
1cc3f7f
Compare
val cachecontrol = Seq("com.typesafe.play" %% "cachecontrol" % cachecontrolVersion) | ||
|
||
val asyncHttpClientVersion = "2.10.0" | ||
val asyncHttpClient = Seq("org.asynchttpclient" % "async-http-client" % asyncHttpClientVersion) | ||
|
||
val akkaVersion = "2.6.0-M2" | ||
val akkaVersion = "2.6.0-M3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in order to have Scala 2.13 support, we must bump the Akka version as well.
Akka 2.6.0-M2 has no binaries for Scala 2.13
project/Dependencies.scala
Outdated
Seq("org.scala-lang.modules" %% "scala-xml" % scalaXmlVersion213) | ||
} else { | ||
Seq("org.scala-lang.modules" %% "scala-xml" % scalaXmlVersion) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will review this part. Seems like unnecessary
Co-authored-by: Marcos Pereira <[email protected]> Co-authored-by: Doug Roper <[email protected]> (cherry picked from commit 8a0eb3c) (cherry picked from commit 1761b49) (cherry picked from commit 9c27797)
954bb9b
to
522f63f
Compare
This is an automated backport of pull request #354 done by Mergify.io
Cherry-pick of 8a0eb3c has failed:
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/