Skip to content
This repository was archived by the owner on Feb 18, 2024. It is now read-only.

Conversation

@zjulbj
Copy link
Contributor

@zjulbj zjulbj commented Dec 2, 2023

No description provided.

@codecov
Copy link

codecov bot commented Dec 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (435f888) 81.31% compared to head (931bf15) 81.29%.

❗ Current head 931bf15 differs from pull request most recent head b388d9e. Consider uploading reports for the commit b388d9e to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #362      +/-   ##
============================================
- Coverage     81.31%   81.29%   -0.02%     
  Complexity       66       66              
============================================
  Files            40       40              
  Lines          1006     1005       -1     
  Branches         90       90              
============================================
- Hits            818      817       -1     
  Misses          137      137              
  Partials         51       51              

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

@zjulbj zjulbj requested a review from lvjing2 December 2, 2023 07:06
Copy link
Collaborator

@lvjing2 lvjing2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lvjing2 lvjing2 merged commit 9a0d943 into master Dec 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants