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

Fix castToDate with format error #3309 #3311

Merged
merged 1 commit into from
Jun 30, 2020
Merged

Conversation

ZivYan
Copy link
Contributor

@ZivYan ZivYan commented Jun 29, 2020

原先用户指定 format 格式的时候未使用导致的问题

@ZivYan ZivYan closed this Jun 29, 2020
@ZivYan
Copy link
Contributor Author

ZivYan commented Jun 29, 2020

#3309

@ZivYan ZivYan reopened this Jun 29, 2020
@codecov-commenter
Copy link

Codecov Report

Merging #3311 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3311   +/-   ##
=========================================
  Coverage     80.48%   80.48%           
- Complexity     7425     7426    +1     
=========================================
  Files           157      157           
  Lines         27793    27793           
  Branches       6685     6685           
=========================================
+ Hits          22369    22370    +1     
+ Misses         3418     3417    -1     
  Partials       2006     2006           
Impacted Files Coverage Δ Complexity Δ
...main/java/com/alibaba/fastjson/util/TypeUtils.java 79.95% <100.00%> (ø) 647.00 <0.00> (+1.00)
...main/java/com/alibaba/fastjson/util/FieldInfo.java 86.44% <0.00%> (ø) 93.00% <0.00%> (-1.00%)
...ava/com/alibaba/fastjson/parser/JSONLexerBase.java 70.00% <0.00%> (+0.03%) 813.00% <0.00%> (+1.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 4f074ed...cd1a4c2. Read the comment docs.

@wenshao wenshao merged commit 13ae28e into alibaba:master Jun 30, 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