Skip to content

Conversation

@funky-eyes
Copy link
Contributor

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

Ⅱ. 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

@funky-eyes funky-eyes added this to the 2.3.0 milestone Oct 9, 2024
@funky-eyes funky-eyes added the type: bug Category issues or prs related to bug. label Oct 9, 2024
@codecov
Copy link

codecov bot commented Oct 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (2.x@a0d0513). Learn more about missing BASE report.
Report is 1 commits behind head on 2.x.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff           @@
##             2.x    #6907   +/-   ##
======================================
  Coverage       ?   52.33%           
  Complexity     ?     6528           
======================================
  Files          ?     1106           
  Lines          ?    38716           
  Branches       ?     4586           
======================================
  Hits           ?    20261           
  Misses         ?    16465           
  Partials       ?     1990           

Copy link
Contributor

@liuqiufeng liuqiufeng left a comment

Choose a reason for hiding this comment

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

LGTM

@funky-eyes funky-eyes merged commit 679f86e into apache:2.x Oct 10, 2024
YvCeung pushed a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Category issues or prs related to bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants