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

fix: Fix problematic error returns #241

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cuishuang
Copy link

If err is not assigned a value here, the default err (must be nil) is returned.

Signed-off-by: cuishuang <imcusg@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Mar 8, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 74.80%. Comparing base (d27f997) to head (c7945ad).

Files with missing lines Patch % Lines
mssql_go19.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   74.91%   74.80%   -0.11%     
==========================================
  Files          32       32              
  Lines        6457     6458       +1     
==========================================
- Hits         4837     4831       -6     
- Misses       1332     1337       +5     
- Partials      288      290       +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.

# 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