Closed sbidoul closed 3 years ago
It should not have pushed the .pot files to this branch...
Merging #48 (ba41392) into 13.0 (a479413) will increase coverage by
11.56%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## 13.0 #48 +/- ##
===========================================
+ Coverage 60.24% 71.81% +11.56%
===========================================
Files 37 36 -1
Lines 488 369 -119
Branches 0 23 +23
===========================================
- Hits 294 265 -29
+ Misses 194 95 -99
- Partials 0 9 +9
Impacted Files | Coverage Δ | |
---|---|---|
.../components/payment_transaction_event_listerner.py | 83.33% <0.00%> (-16.67%) |
:arrow_down: |
shopinvader_payment/models/payment_transaction.py | 75.00% <0.00%> (-12.50%) |
:arrow_down: |
.../components/payment_transaction_event_listerner.py | 83.33% <0.00%> (-8.34%) |
:arrow_down: |
invader_payment/models/payment_transaction.py | 79.16% <0.00%> (-8.34%) |
:arrow_down: |
...nvader_payment/services/invader_payment_service.py | 84.61% <0.00%> (-7.70%) |
:arrow_down: |
...nvader_payment/services/invader_payment_service.py | 78.57% <0.00%> (-7.15%) |
:arrow_down: |
test_invader_payment/tests/common.py | ||
test_invader_payment/models/res_partner.py | ||
shopinvader_payment/models/sale_order.py | 83.33% <0.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update a479413...ba41392. Read the comment docs.
/ocabot merge nobump
This PR looks fantastic, let's merge it! Prepared branch 13.0-ocabot-merge-pr-48-by-sbidoul-bump-nobump, awaiting test results.
So. It correctey generated pot files to the ocabot merge branch: https://github.com/shopinvader/odoo-shopinvader-payment/compare/13.0...13.0-ocabot-merge-pr-48-by-sbidoul-bump-nobump. But, that did not trigger a GitHub action so the bot will never merge... sigh.
/ocabot merge nobump
On my way to merge this fine PR! Prepared branch 13.0-ocabot-merge-pr-48-by-sbidoul-bump-nobump, awaiting test results.
@sbidoul your merge command was aborted due to failed check(s), which you can inspect on this commit of 13.0-ocabot-merge-pr-48-by-sbidoul-bump-nobump.
After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.
/ocabot merge nobump
On my way to merge this fine PR! Prepared branch 13.0-ocabot-merge-pr-48-by-sbidoul-bump-nobump, awaiting test results.
Congratulations, your PR was merged at 3861aad82739a9de307c0bf2e5adcf61b8790313. Thanks a lot for contributing to shopinvader. ❤️
\o/
Use the latest template