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

capture: do not set logging.raiseExceptions = False #4943

Merged

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Mar 18, 2019

Ref: #4942

@codecov
Copy link

codecov bot commented Mar 18, 2019

Codecov Report

Merging #4943 into features will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           features    #4943      +/-   ##
============================================
+ Coverage     96.06%   96.06%   +<.01%     
============================================
  Files           114      114              
  Lines         25745    25741       -4     
  Branches       2548     2547       -1     
============================================
- Hits          24731    24728       -3     
  Misses          704      704              
+ Partials        310      309       -1
Impacted Files Coverage Δ
src/_pytest/capture.py 95.14% <ø> (+0.17%) ⬆️

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 1c9dcf1...08ded29. Read the comment docs.

@nicoddemus
Copy link
Member

Sorry for the delay!

Hmmm are you getting an error because of this code? We want to add a CHANGELOG error explaining in detail why we are reverting this code, and get a fix back quickly if this issue comes back.

@blueyed
Copy link
Contributor Author

blueyed commented Mar 22, 2019

Hmmm are you getting an error because of this code?

No, just found it confusing - likely when looking at coverage reports.

I'll add a changelog.

@blueyed
Copy link
Contributor Author

blueyed commented Mar 23, 2019

Maybe related to #4988.

@blueyed blueyed added this to the 5.0 milestone Mar 26, 2019
@nicoddemus
Copy link
Member

Would you like to include this one in 4.4 @blueyed?

@blueyed
Copy link
Contributor Author

blueyed commented Mar 29, 2019

Rather not.. too much in there already.. ;)

@nicoddemus
Copy link
Member

Gentle ping in case you want to pick this up again. 👍

@blueyed blueyed force-pushed the remove-silence_logging_at_shutdown branch from 3fdaa58 to 08ded29 Compare April 2, 2019 11:20
@blueyed
Copy link
Contributor Author

blueyed commented Apr 2, 2019

Added changelog, and rebased it.

@nicoddemus
Copy link
Member

I noticed you added this to the 5.0 milestone, is that correct?

@blueyed
Copy link
Contributor Author

blueyed commented Apr 2, 2019

Yes, before - but only since it was the next after 4.4.

@nicoddemus
Copy link
Member

OK to move to 4.5 then?

@blueyed
Copy link
Contributor Author

blueyed commented Apr 2, 2019

Sure, I'd just merge it already.

@blueyed blueyed modified the milestones: 5.0, 4.5 Apr 2, 2019
@nicoddemus nicoddemus merged commit 15ea5ce into pytest-dev:features Apr 3, 2019
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.

2 participants