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 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cb6f718) 82.35% compared to head (3037cb2) 82.35%.
Report is 9 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #311   +/-   ##
=========================================
  Coverage     82.35%   82.35%           
  Complexity      131      131           
=========================================
  Files            38       38           
  Lines           918      918           
  Branches         77       77           
=========================================
  Hits            756      756           
  Misses          115      115           
  Partials         47       47           

☔ 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

Copy link
Collaborator

@yuanyuancin yuanyuancin left a comment

Choose a reason for hiding this comment

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

LGFM

@hustchaya hustchaya merged commit 2d262a4 into master Nov 20, 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.

4 participants