Skip to content
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

Addressed #16 #18

Merged
merged 1 commit into from
Jun 18, 2016
Merged

Addressed #16 #18

merged 1 commit into from
Jun 18, 2016

Conversation

nrktkt
Copy link
Member

@nrktkt nrktkt commented Jun 18, 2016

#16

Changed optional fields in HalLink to use java Optional.
Added HalLinkSerializer to fix odd issue where Optional is not serialized correctly despite jackson module.
Made indentation uniform

@nrktkt nrktkt force-pushed the feature/#16_optional branch from 7a7df76 to 0286ad5 Compare June 18, 2016 05:44
@codecov-io
Copy link

codecov-io commented Jun 18, 2016

Current coverage is 85.37%

Merging #18 into master will increase coverage by 4.01%

@@             master        #18   diff @@
==========================================
  Files             7          7          
  Lines           236        253    +17   
  Methods           0          0          
  Messages          0          0          
  Branches         33         35     +2   
==========================================
+ Hits            192        216    +24   
+ Misses           34         31     -3   
+ Partials         10          6     -4   

Powered by Codecov. Last updated by 4b37a4a...b22d761

@nrktkt nrktkt force-pushed the feature/#16_optional branch 3 times, most recently from ad6e068 to b22d761 Compare June 18, 2016 06:01
Changed optional fields in HalLink to use java Optional.
Added HalLinkSerializer to fix odd issue where Optional is not serialized correctly despite jackson module.
Made indentation uniform
@nrktkt nrktkt merged commit 1ff13f2 into master Jun 18, 2016
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.

2 participants