Closed justin-tay closed 3 weeks ago
Attention: Patch coverage is 64.28571%
with 15 lines
in your changes missing coverage. Please review.
Project coverage is 78.68%. Comparing base (
48ca3c2
) to head (8953fe8
). Report is 31 commits behind head on master.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Adds an
JsonNodeReader
interface to abstract out the logic for reading content into aJsonNode
.This will make it easier to customize how the reading is done with a custom implementation. This also potentially allows easier reuse of the logic of reading location aware nodes by trying to decouple this from the
JsonSchemaFactory
.This deprecates the setting of
jsonMapper
andyamlMapper
on theJsonSchemaFactory
in favor of thejsonNodeReader
.