Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

enhance code quality #172

Merged
merged 1 commit into from
May 7, 2024

Conversation

River-sh
Copy link
Contributor

@River-sh River-sh commented May 6, 2024

Improve code quality

@River-sh River-sh force-pushed the enhance/optimize_raven_agent branch 3 times, most recently from 6d53b3a to 4139f39 Compare May 6, 2024 11:19
@River-sh River-sh force-pushed the enhance/optimize_raven_agent branch from 4139f39 to 2c4a314 Compare May 6, 2024 11:26
Copy link

codecov bot commented May 6, 2024

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 36.13%. Comparing base (ab59000) to head (2c4a314).
Report is 3 commits behind head on main.

Files Patch % Lines
pkg/utils/utils.go 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
+ Coverage   35.85%   36.13%   +0.27%     
==========================================
  Files          12       12              
  Lines        1570     1558      -12     
==========================================
  Hits          563      563              
+ Misses        893      881      -12     
  Partials      114      114              
Flag Coverage Δ
unittests 36.13% <0.00%> (+0.27%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@njucjc njucjc merged commit 8add760 into openyurtio:main May 7, 2024
10 of 11 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants