Skip to content

Conversation

gertjana
Copy link
Member

… schema

@coveralls
Copy link

Coverage Status

Coverage remained the same at 53.21% when pulling c4cad1e on TNM-4930 into 1a62e0f on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.7%) to 54.878% when pulling 0c8a4b1 on TNM-4930 into 1a62e0f on master.

Copy link
Contributor

@reinierl reinierl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work, merging it in. I thought the dependency on Jackson was better off with a "% test" as long as we use it only for running the tests, so I changed that.

build.sbt Outdated
val slf4jV = "1.7.12"

val json4sNative = "org.json4s" %% "json4s-native" % json4sV
val json4sjackson = "org.json4s" %% "json4s-jackson" % json4sV
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this for some specific reason? Otherwise we could just as well import org.json4s.native.JsonMethods instead of org.json4s.jackson.JsonMethods, and get rid of one dependency.

@reinierl reinierl merged commit 75b215d into master Oct 16, 2017
@reinierl reinierl deleted the TNM-4930 branch October 16, 2017 14:19
@coveralls
Copy link

Coverage Status

Coverage increased (+1.7%) to 54.878% when pulling 98e0dbc on TNM-4930 into 1a62e0f on master.

@gertjana
Copy link
Member Author

gertjana commented Oct 16, 2017

the dependency on jackson is needed as the java json validator library uses
com.fasterxml.jackson.databind.JsonNode as input parameter types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants