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

support map and multi slice #43

Merged
merged 3 commits into from
Apr 7, 2024
Merged

support map and multi slice #43

merged 3 commits into from
Apr 7, 2024

Conversation

zc2638
Copy link
Owner

@zc2638 zc2638 commented Apr 3, 2024

No description provided.

Copy link

codecov bot commented Apr 3, 2024

Codecov Report

Attention: Patch coverage is 92.10526% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 90.44%. Comparing base (35fc1f3) to head (c84e129).

Files Patch % Lines
reflect.go 86.76% 8 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   89.73%   90.44%   +0.71%     
==========================================
  Files          11       13       +2     
  Lines         945      827     -118     
==========================================
- Hits          848      748     -100     
+ Misses         85       72      -13     
+ Partials       12        7       -5     
Flag Coverage Δ
all 90.44% <92.10%> (+0.71%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@zc2638
Copy link
Owner Author

zc2638 commented Apr 3, 2024

I refactored part of the parsing code, and now it works well with maps and arrays without restrictions on levels.
If you are interested, you can test it to see if there are any other problems.
@HugoWw @TianTianBigWang

@zc2638 zc2638 merged commit 9b1f0c8 into main Apr 7, 2024
4 checks passed
@zc2638 zc2638 deleted the feat/parse branch April 7, 2024 01:31
@HugoWw
Copy link
Contributor

HugoWw commented Apr 7, 2024

I refactored part of the parsing code, and now it works well with maps and arrays without restrictions on levels. If you are interested, you can test it to see if there are any other problems. @HugoWw @TianTianBigWang

ok

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.

2 participants