Skip to content

Add Retrofit2ConverterFactory for compatibility with fastjson 1.x.#47

Merged
wenshao merged 3 commits intoalibaba:mainfrom
VictorZeng:main
Apr 22, 2022
Merged

Add Retrofit2ConverterFactory for compatibility with fastjson 1.x.#47
wenshao merged 3 commits intoalibaba:mainfrom
VictorZeng:main

Conversation

@VictorZeng
Copy link
Copy Markdown
Collaborator

@VictorZeng VictorZeng commented Apr 22, 2022

Add Retrofit2ConverterFactory for compatibility with fastjson 1.x.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 22, 2022

Codecov Report

Attention: Patch coverage is 54.28571% with 32 lines in your changes missing coverage. Please review.

Project coverage is 64.64%. Comparing base (ebdb322) to head (7182b24).
Report is 4278 commits behind head on main.

Files with missing lines Patch % Lines
...libaba/fastjson/support/config/FastJsonConfig.java 30.76% 27 Missing ⚠️
...on/support/retrofit/Retrofit2ConverterFactory.java 82.75% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main      #47      +/-   ##
============================================
- Coverage     64.65%   64.64%   -0.01%     
- Complexity     7902     7914      +12     
============================================
  Files           407      409       +2     
  Lines         44971    45041      +70     
  Branches       9105     9106       +1     
============================================
+ Hits          29074    29118      +44     
- Misses        12361    12385      +24     
- Partials       3536     3538       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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