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

Extract the version properties for fastjson 1.x #59

Merged
merged 5 commits into from
Apr 23, 2022

Conversation

VictorZeng
Copy link
Collaborator

@VictorZeng VictorZeng commented Apr 23, 2022

Extract the version properties for fastjson 1.x @oldratlee PTAL~

@codecov-commenter
Copy link

codecov-commenter commented Apr 23, 2022

Codecov Report

Merging #59 (968e490) into main (67e48ae) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #59   +/-   ##
=========================================
  Coverage     64.92%   64.92%           
+ Complexity     8066     8062    -4     
=========================================
  Files           420      420           
  Lines         45418    45418           
  Branches       9167     9168    +1     
=========================================
  Hits          29488    29488           
  Misses        12375    12375           
  Partials       3555     3555           
Impacted Files Coverage Δ
...com/alibaba/fastjson2/util/ApacheLang3Support.java 0.00% <0.00%> (ø)
...ibaba/fastjson2/writer/ObjectWriterBaseModule.java 79.39% <0.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 67e48ae...968e490. Read the comment docs.

@wenshao
Copy link
Member

wenshao commented Apr 23, 2022

fastjson.version改名为fastjson1x.version是不是更合适?

@VictorZeng
Copy link
Collaborator Author

fastjson.version改名为fastjson1x.version是不是更合适?

好的 稍后我会再提交

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