Skip to content
This repository was archived by the owner on Oct 23, 2024. It is now read-only.

修复在FastJsonHttpMessageConverter 中使用了spring4.x版本新增的TypeResolvable 导致sp…#1501

Merged
wenshao merged 1 commit intoalibaba:masterfrom
neil4dong:master
Sep 29, 2017
Merged

修复在FastJsonHttpMessageConverter 中使用了spring4.x版本新增的TypeResolvable 导致sp…#1501
wenshao merged 1 commit intoalibaba:masterfrom
neil4dong:master

Conversation

@neil4dong
Copy link
Copy Markdown
Contributor

…ring3.x 不可用的问题。

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1501 into master will decrease coverage by 0.01%.
The diff coverage is 45.94%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1501      +/-   ##
============================================
- Coverage     82.78%   82.76%   -0.02%     
+ Complexity     6015     6010       -5     
============================================
  Files           136      136              
  Lines         22008    22019      +11     
  Branches       5021     5023       +2     
============================================
+ Hits          18220    18225       +5     
- Misses         2548     2551       +3     
- Partials       1240     1243       +3
Impacted Files Coverage Δ Complexity Δ
...n/support/spring/FastJsonHttpMessageConverter.java 73.46% <45.94%> (-1.54%) 35 <1> (-5)
...com/alibaba/fastjson/parser/JSONReaderScanner.java 72.61% <0%> (-1.28%) 44% <0%> (-1%)
...ava/com/alibaba/fastjson/parser/JSONLexerBase.java 71.52% <0%> (+0.03%) 785% <0%> (+1%) ⬆️

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 b55cdd4...fe8c660. Read the comment docs.

@wenshao wenshao merged commit 6b9bc44 into alibaba:master Sep 29, 2017
@VictorZeng VictorZeng mentioned this pull request Sep 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants