Skip to content
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

Merge master #44

Merged
merged 56 commits into from
Nov 1, 2017
Merged

Merge master #44

merged 56 commits into from
Nov 1, 2017

Conversation

ticosax
Copy link

@ticosax ticosax commented Nov 1, 2017

No description provided.

blueyed and others added 30 commits October 6, 2017 21:32
"pragma: no cover" is used in newer code already, and not being
ignored/handled correctly.

This also removes "coverage: omit" for where we have actual coverage,
when merging reports.
Set related name to avoid conflict with DUA
log_exception / EventProcessingException expects a string for `data`.
…-captured

tests: improve/harden test for captured queries
paltman and others added 26 commits October 30, 2017 09:35
coverage: use default exclude_lines and clean up
pinax/stripe/tests/settings: fix deprecation warnings
pinax/stripe/tests/settings: remove support for Django < 1.8
actions.charges.create: require customer or source
add_event: get stripe_account from message
…t-data

test_webhook_duplicate_event: add assertion for data
…-lazy-stripe_customer

CustomerSubscriptionWebhook: let sync_customer fetch the live customer
…ge-balance

Fix type checking of balance_transaction
…hable_key"

This reverts commit 7881f72, reversing
changes made to f0172f1.
@codecov
Copy link

codecov bot commented Nov 1, 2017

Codecov Report

Merging #44 into next will decrease coverage by 0.21%.
The diff coverage is 76.92%.

Impacted file tree graph

@@            Coverage Diff             @@
##             next      #44      +/-   ##
==========================================
- Coverage   99.06%   98.84%   -0.22%     
==========================================
  Files          34       34              
  Lines        1813     1825      +12     
  Branches      160      162       +2     
==========================================
+ Hits         1796     1804       +8     
- Misses          9       13       +4     
  Partials        8        8
Flag Coverage Δ
#py36dj111psql 98.84% <76.92%> (-0.22%) ⬇️
#py36dj20psql 98.84% <76.92%> (-0.22%) ⬇️
Impacted Files Coverage Δ
pinax/stripe/middleware.py 73.91% <0%> (-26.09%) ⬇️
pinax/stripe/models.py 100% <100%> (ø) ⬆️
pinax/stripe/webhooks.py 100% <100%> (ø) ⬆️
pinax/stripe/management/commands/init_customers.py 100% <100%> (ø) ⬆️
pinax/stripe/views.py 100% <100%> (ø) ⬆️
pinax/stripe/actions/charges.py 100% <100%> (ø) ⬆️
pinax/stripe/actions/events.py 100% <100%> (ø) ⬆️
pinax/stripe/forms.py 100% <0%> (+1.92%) ⬆️

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 888303a...ac40f67. Read the comment docs.

@ticosax ticosax merged commit ec993ef into lock8:next Nov 1, 2017
@ticosax ticosax deleted the merge-master branch November 1, 2017 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants