-
Notifications
You must be signed in to change notification settings - Fork 5
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
Do transactions correctly #285
Conversation
I saw this test fail again in the CI (only for postgres):
So it seems this PR does not actually fix that.... However it's still the correct way to use transactions in |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #285 +/- ##
==========================================
+ Coverage 93.35% 93.37% +0.01%
==========================================
Files 63 63
Lines 5180 5179 -1
==========================================
Hits 4836 4836
+ Misses 344 343 -1
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. |
Hmm, the postgres tests are failing painfully....
|
And it looks like they're running three times..? that seems weird! |
* main: Update breaking API calls for new `p2panda-rs` version (#293) Update Cargo.lock as well Use released p2panda-rs version 0.7.0 Migrate CLI from `structopt` to `clap` (#289) Increase timeout for failing materializer test Introduce requeue flag (#286) Do transactions correctly (#285) Add libp2p service and configuration (#282)
closes: #283
📋 Checklist
CHANGELOG.md