Skip to content

fix: return rows.Close() err#164

Merged
ginokent merged 1 commit intomainfrom
fix-database-sql-rows-close
Jan 12, 2024
Merged

fix: return rows.Close() err#164
ginokent merged 1 commit intomainfrom
fix-database-sql-rows-close

Conversation

@ginokent
Copy link
Member

Ticket / Issue Number

Note
Please fill in the ticket or issue number.

Example:

#3

What's changed

Note
Please explain what changes this pull request will make.

Example:

  • Added functionality to perform 'bar' on 'foo'.

Check List

  • Assign labels
  • Add appropriate test cases

Remark

Note
Please provide additional remarks if necessary.

@github-actions github-actions bot added the fix A bug fix label Jan 12, 2024
@codecov
Copy link

codecov bot commented Jan 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6f7e5bc) 100.00% compared to head (d8a32bf) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #164   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           79        79           
  Lines         3830      3836    +6     
=========================================
+ Hits          3830      3836    +6     

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

@ginokent ginokent force-pushed the fix-database-sql-rows-close branch from 1a970b3 to d8a32bf Compare January 12, 2024 14:43
@ginokent ginokent marked this pull request as ready for review January 12, 2024 14:45
@ginokent ginokent requested a review from a team as a code owner January 12, 2024 14:45
@ginokent ginokent merged commit 2931209 into main Jan 12, 2024
@ginokent ginokent deleted the fix-database-sql-rows-close branch January 12, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants