Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@WyRainBow
Copy link
Contributor

fix #992

@github-actions github-actions bot added bug Something isn't working coding labels Nov 23, 2025
@codecov-commenter
Copy link

codecov-commenter commented Nov 23, 2025

Codecov Report

❌ Patch coverage is 57.50000% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.82%. Comparing base (b18bcb0) to head (9abd367).

Files with missing lines Patch % Lines
pkg/datasource/sql/datasource/base/meta_cache.go 48.38% 15 Missing and 1 partial ⚠️
pkg/datasource/sql/at_resource_manager.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #996      +/-   ##
==========================================
+ Coverage   56.49%   56.82%   +0.33%     
==========================================
  Files         266      266              
  Lines       17484    17494      +10     
==========================================
+ Hits         9877     9941      +64     
+ Misses       6772     6720      -52     
+ Partials      835      833       -2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@WyRainBow WyRainBow force-pushed the feature/fix-goroutine-graceful-exit branch 9 times, most recently from c6222bb to 0515f52 Compare November 23, 2025 16:17
@WyRainBow WyRainBow force-pushed the feature/fix-goroutine-graceful-exit branch from 0515f52 to 7623f48 Compare November 23, 2025 16:26
@WyRainBow
Copy link
Contributor Author

Perhaps you could clone the samples repository and replace its "seata go" dependency with your modified local version to verify if it works correctly.

Verified by cloning apache/seata-go-samples, replacing the dependency with my local incubator-seata-go via go mod edit -replace ..., running go mod tidy, and all go test ./... runs succeeded on both the samples repo and the main repo.

@tew-axiom
Copy link
Contributor

Plz check and resolve the conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working coding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants