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

[v3.x] Migrate json support from net.sf.json-lib to jackson #98

Merged
merged 25 commits into from
May 24, 2023

Conversation

seppinho
Copy link
Member

@seppinho seppinho commented May 6, 2023

  • Migrate from outdated son library to jackson lib
  • Create data transfer objects
  • Add counters

@lukfor lukfor changed the title Feature/jackson integration Migrate json support from net.sf.json-lib to jackson May 7, 2023
@lukfor lukfor changed the title Migrate json support from net.sf.json-lib to jackson [v3.x] Migrate json support from net.sf.json-lib to jackson May 7, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 8, 2023

Codecov Report

Merging #98 (93e1657) into micronaut (ffeba16) will increase coverage by 1.06%.
The diff coverage is 57.22%.

@@               Coverage Diff               @@
##             micronaut      #98      +/-   ##
===============================================
+ Coverage        43.20%   44.27%   +1.06%     
- Complexity        1137     1232      +95     
===============================================
  Files              156      163       +7     
  Lines             8431     8738     +307     
  Branches           946      952       +6     
===============================================
+ Hits              3643     3869     +226     
- Misses            4537     4618      +81     
  Partials           251      251              
Impacted Files Coverage Δ
...ene/mapred/plugins/nextflow/NextflowCollector.java 0.00% <0.00%> (ø)
...udgene/mapred/plugins/nextflow/NextflowPlugin.java 64.28% <ø> (ø)
...dgene/mapred/plugins/nextflow/NextflowProcess.java 0.00% <0.00%> (ø)
...loudgene/mapred/plugins/nextflow/NextflowStep.java 0.00% <0.00%> (ø)
...loudgene/mapred/plugins/nextflow/NextflowTask.java 0.00% <0.00%> (ø)
...udgene/mapred/server/controller/AppController.java 0.00% <0.00%> (ø)
...ene/mapred/server/controller/ImportController.java 0.00% <ø> (ø)
...e/mapred/server/controller/JobAdminController.java 0.00% <0.00%> (ø)
...e/mapred/server/controller/NextflowController.java 0.00% <0.00%> (ø)
...apred/server/controller/ServerAdminController.java 0.00% <0.00%> (ø)
... and 17 more

... and 9 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@seppinho seppinho merged commit 3b874d1 into micronaut May 24, 2023
@seppinho seppinho deleted the feature/jackson-integration branch May 24, 2023 12:10
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