-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update dependencies #184
Update dependencies #184
Conversation
project/Dependencies.scala
Outdated
} | ||
} | ||
|
||
def monocle(module: String) = Def.setting { |
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.
this will probably continue to be unused for gt-server (hopefully) I'd leave it out but meh.
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.
oh oops, good catch
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.
LGTM
@@ -42,6 +74,8 @@ object Dependencies { | |||
val http4sCirce = "org.http4s" %% "http4s-circe" % http4sVer | |||
val http4sDsl = "org.http4s" %% "http4s-dsl" % http4sVer | |||
val http4sXml = "org.http4s" %% "http4s-scala-xml" % http4sVer | |||
|
|||
val jaxbApi = "javax.xml.bind" % "jaxb-api" % jaxbApiVer |
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.
Added so that project compiles on Java 11
Overview
This PR splits the cats ecosystem dependencies to match GeoTrellis 3.2
Checklist
Testing Instructions
evicted
from thesbt
shell too in order to make sure that between versions cats dependencies looked sane, but there's a lot of evictions