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

Conversation

@hustchaya
Copy link
Collaborator

Motivation

Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.

Modification

Describe the idea and modifications you've done.

Result

Resolved or fixed #.

If there is no issue then describe the changes introduced by this PR.

@codecov
Copy link

codecov bot commented Nov 29, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (dcebc07) 76.58% compared to head (c120035) 81.29%.
Report is 6 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #350      +/-   ##
============================================
+ Coverage     76.58%   81.29%   +4.70%     
- Complexity        0       66      +66     
============================================
  Files            17       40      +23     
  Lines          1610     1005     -605     
  Branches          0       90      +90     
============================================
- Hits           1233      817     -416     
+ Misses          273      137     -136     
+ Partials        104       51      -53     

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

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 610e30a into master Nov 30, 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