Skip to content

Conversation

kvii
Copy link
Contributor

@kvii kvii commented Jan 31, 2024

Description (what this PR does / why we need it):

Fix test failed.

Which issue(s) this PR fixes (resolves / be part of):

fixes/resolves #3181

Other special notes for the reviewers:

Forget to pull upstream before release a PR. Do It again.

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jan 31, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a556a2b) 84.62% compared to head (4239d4f) 84.62%.

❗ Current head 4239d4f differs from pull request most recent head 4ed7aa5. Consider uploading reports for the commit 4ed7aa5 to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3183   +/-   ##
=======================================
  Coverage   84.62%   84.62%           
=======================================
  Files          88       88           
  Lines        3993     3993           
=======================================
  Hits         3379     3379           
  Misses        440      440           
  Partials      174      174           

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

@dosubot dosubot bot added the LGTM label Feb 2, 2024
@hawkingrei hawkingrei merged commit d9f390c into go-kratos:main Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LGTM size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test failed on different time zone.
4 participants