Skip to content

Follow up(#317) Compatible with Jackson 2.9.x#320

Merged
stevehu merged 1 commit intonetworknt:masterfrom
pan3793:317
Aug 11, 2020
Merged

Follow up(#317) Compatible with Jackson 2.9.x#320
stevehu merged 1 commit intonetworknt:masterfrom
pan3793:317

Conversation

@pan3793
Copy link
Copy Markdown
Contributor

@pan3793 pan3793 commented Aug 11, 2020

Fix compatible issue with Jackson 2.9.x introduced by 3a3e1a8 d5a861a .

It should pass compile and test with Jackson 2.9.x, 2.10.x 2.11.x.

You can verify by:

mvn -Dversion.jackson=2.9.0 clean test
mvn -Dversion.jackson=2.10.0 clean test
mvn -Dversion.jackson=2.11.0 clean test

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #320 into master will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #320   +/-   ##
=========================================
  Coverage     70.80%   70.80%           
  Complexity      652      652           
=========================================
  Files            75       75           
  Lines          2497     2497           
  Branches        529      529           
=========================================
  Hits           1768     1768           
  Misses          525      525           
  Partials        204      204           
Impacted Files Coverage Δ Complexity Δ
...a/com/networknt/schema/PropertyNamesValidator.java 86.66% <0.00%> (ø) 12.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d8e6ac...924d95f. Read the comment docs.

@stevehu stevehu merged commit f841582 into networknt:master Aug 11, 2020
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