Skip to content

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

Merged
nicoddemus merged 1 commit into
pytest-dev:featuresfrom
blueyed:remove-silence_logging_at_shutdown
Apr 3, 2019
Merged

capture: do not set logging.raiseExceptions = False#4943
nicoddemus merged 1 commit into
pytest-dev:featuresfrom
blueyed:remove-silence_logging_at_shutdown

Conversation

@blueyed

@blueyed blueyed commented Mar 18, 2019

Copy link
Copy Markdown
Contributor

Ref: #4942

@codecov

codecov Bot commented Mar 18, 2019

Copy link
Copy Markdown

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
Copy Markdown
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

blueyed commented Mar 22, 2019

Copy link
Copy Markdown
Contributor Author

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

blueyed commented Mar 23, 2019

Copy link
Copy Markdown
Contributor Author

Maybe related to #4988.

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

Copy link
Copy Markdown
Member

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

@blueyed

blueyed commented Mar 29, 2019

Copy link
Copy Markdown
Contributor Author

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

@nicoddemus

Copy link
Copy Markdown
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

blueyed commented Apr 2, 2019

Copy link
Copy Markdown
Contributor Author

Added changelog, and rebased it.

@nicoddemus

Copy link
Copy Markdown
Member

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

@blueyed

blueyed commented Apr 2, 2019

Copy link
Copy Markdown
Contributor Author

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

@nicoddemus

Copy link
Copy Markdown
Member

OK to move to 4.5 then?

@blueyed

blueyed commented Apr 2, 2019

Copy link
Copy Markdown
Contributor Author

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