Closed PierrickBrun closed 3 years ago
@PierrickBrun Could you retry travis ?
Merging #28 (4894295) into 12.0 (c27a6f6) will decrease coverage by
3.11%
. The diff coverage is36.73%
.:exclamation: Current head 4894295 differs from pull request most recent head 9f316ea. Consider uploading reports for the commit 9f316ea to get more accurate results
@@ Coverage Diff @@
## 12.0 #28 +/- ##
==========================================
- Coverage 85.30% 82.18% -3.12%
==========================================
Files 55 57 +2
Lines 694 741 +47
Branches 39 42 +3
==========================================
+ Hits 592 609 +17
- Misses 90 118 +28
- Partials 12 14 +2
Impacted Files | Coverage Δ | |
---|---|---|
invader_payment_stripe/services/payment_stripe.py | 63.46% <22.22%> (-13.76%) |
:arrow_down: |
...nvader_payment/services/invader_payment_service.py | 87.50% <33.33%> (-12.50%) |
:arrow_down: |
.../components/payment_transaction_event_listerner.py | 83.33% <50.00%> (-3.04%) |
:arrow_down: |
invader_payment_stripe/models/payment.py | 56.25% <56.25%> (ø) |
|
invader_payment_stripe/models/__init__.py | 100.00% <100.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 c27a6f6...9f316ea. Read the comment docs.
This PR has the approved
label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
/ocabot merge minor
Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 12.0-ocabot-merge-pr-28-by-hparfr-bump-minor, awaiting test results.
Congratulations, your PR was merged at b93542e7b39f24d0b486b6b51d4a60e73e549ac8. Thanks a lot for contributing to shopinvader. ❤️
Codecov Report
71.15% <29.62%> (-14.93%)
87.50% <50.00%> (-3.41%)
56.25% <56.25%> (ø)
93.75% <66.66%> (-6.25%)
Continue to review full report at Codecov.