-
-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: add support for transactions #427
Conversation
this is a WIP as there is currently no CLI binary available for us so we cannot upgrade to the latest prisma version, however I have built the CLI locally and have been testing with that. also some of the tests are broken
for more information, see https://pre-commit.ci
Codecov ReportBase: 94.32% // Head: 36.78% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #427 +/- ##
===========================================
- Coverage 94.32% 36.78% -57.54%
===========================================
Files 120 113 -7
Lines 6059 6002 -57
Branches 1037 457 -580
===========================================
- Hits 5715 2208 -3507
- Misses 301 3715 +3414
- Partials 43 79 +36
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Any news? |
@listepo this is already supported, this PR is old and outdated. https://prisma-client-py.readthedocs.io/en/stable/reference/transactions/ |
Opening prematurely to track status of tests / linters. When I have to occasionally update this branch I forget which things are "intended" to fail at the moment, while it is easy to just reset in git, it's nice to have it here too.
closes #53
TODO:
rollback()
on error