search

matheusgomes28 / urchin

Headless CMS written in Golang
MIT License
70 stars 21 forks source link

Fix Transactions Rollback #59

Closed matheusgomes28 closed 7 months ago

matheusgomes28 commented 7 months ago

In a normal case where all the database calls in this PR succeed, the defer will throw an error as the transactions are already committed.

codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 34.83%. Comparing base (1672494) to head (f100aa8).

Files Patch % Lines
database/database.go 0.00% 6 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #59 +/- ## ========================================== + Coverage 34.64% 34.83% +0.19% ========================================== Files 15 15 Lines 713 709 -4 ========================================== Hits 247 247 + Misses 440 436 -4 Partials 26 26 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.