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] 修复JSONPath.set()空指针问题 #2146

Merged
merged 2 commits into from
Jan 2, 2024

Conversation

mimajiushi
Copy link
Contributor

What this PR does / why we need it?

@CLAassistant
Copy link

CLAassistant commented Jan 1, 2024

CLA assistant check
All committers have signed the CLA.

@wenshao
Copy link
Member

wenshao commented Jan 2, 2024

请签署CLA

@mimajiushi
Copy link
Contributor Author

请签署CLA

没看懂怎么签,我看早期PR没有,或者你修了把这个PR关掉吧

@codecov-commenter
Copy link

codecov-commenter commented Jan 2, 2024

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (67e1e74) 75.37% compared to head (c1b371e) 75.36%.
Report is 1 commits behind head on main.

Files Patch % Lines
...in/java/com/alibaba/fastjson2/JSONPathSegment.java 16.66% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2146      +/-   ##
============================================
- Coverage     75.37%   75.36%   -0.01%     
- Complexity    21637    21642       +5     
============================================
  Files           634      634              
  Lines         85368    85383      +15     
  Branches      18920    18928       +8     
============================================
+ Hits          64343    64351       +8     
- Misses        13109    13111       +2     
- Partials       7916     7921       +5     

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

@wenshao wenshao merged commit 6e0d5ca into alibaba:main Jan 2, 2024
4 checks passed
@wenshao
Copy link
Member

wenshao commented Jan 2, 2024

已经合入,欢迎以后继续参与进来!

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.

4 participants