-
Notifications
You must be signed in to change notification settings - Fork 541
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
Add dict support on metadata, optimize datetime format judgment rules, add eq for combiner #135
Conversation
for more information, see https://pre-commit.ci
still draft
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #135 +/- ##
==========================================
+ Coverage 80.26% 80.38% +0.11%
==========================================
Files 66 66
Lines 2975 3008 +33
==========================================
+ Hits 2388 2418 +30
- Misses 587 590 +3 ☔ View full report in Codecov by Sentry. |
Co-authored-by: Zhongsheng Ji <9573586@qq.com>
I had a breakfast before fix the code, I found that @Wh1isper had already correct them when I push commits. hhhhhh, happy new year! |
Happy new year~ |
Description
This PR mainly completes:
How has this been tested?
The content of this modification has been covered by test cases.
Types of changes
Checklist: