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

optimize: remove dependency without license #5889

Merged
merged 4 commits into from
Sep 28, 2023

Conversation

slievrly
Copy link
Member

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

remove dependency without license

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

Merging #5889 (d2b9819) into develop (539463c) will not change coverage.
The diff coverage is n/a.

❗ Current head d2b9819 differs from pull request most recent head 83b5b4e. Consider uploading reports for the commit 83b5b4e to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             develop    #5889   +/-   ##
==========================================
  Coverage      48.78%   48.78%           
- Complexity      4135     4137    +2     
==========================================
  Files            777      777           
  Lines          27944    27944           
  Branches        3412     3412           
==========================================
  Hits           13633    13633           
- Misses         12889    12890    +1     
+ Partials        1422     1421    -1     

see 2 files with indirect coverage changes

Signed-off-by: slievrly <slievrly@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Finished
Development

Successfully merging this pull request may close these issues.

1 participant