Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

fix force payment concent integration test #4989

Merged
merged 3 commits into from
Dec 11, 2019

Conversation

shadeofblue
Copy link
Contributor

additionally

  • don't send SystemExit and Exit to sentry in any case
  • don't try to freeze raven's time calls

* don't send `SystemExit` and `Exit` to sentry in any case
* don't try to freeze raven's time calls
Copy link
Contributor

@maaktweluit maaktweluit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

small nit and bug in the log message

golem/ethereum/incomeskeeper.py Outdated Show resolved Hide resolved
@shadeofblue shadeofblue force-pushed the 0.22-fix-force-payment-concent-integration-test branch from ce25016 to dd72400 Compare December 11, 2019 08:22
@codecov
Copy link

codecov bot commented Dec 11, 2019

Codecov Report

Merging #4989 into b0.22 will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##            b0.22    #4989      +/-   ##
==========================================
- Coverage   90.12%   90.12%   -0.01%     
==========================================
  Files         233      233              
  Lines       21833    21834       +1     
==========================================
  Hits        19677    19677              
- Misses       2156     2157       +1

@jiivan jiivan merged commit 5dc6b51 into b0.22 Dec 11, 2019
@jiivan jiivan deleted the 0.22-fix-force-payment-concent-integration-test branch December 11, 2019 11:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants