Skip to content

Do not treat warnings as errors during pytest_configure.#5729

Merged
nicoddemus merged 1 commit intopytest-dev:masterfrom
Stranger6667:issue-5115
Aug 15, 2019
Merged

Do not treat warnings as errors during pytest_configure.#5729
nicoddemus merged 1 commit intopytest-dev:masterfrom
Stranger6667:issue-5115

Conversation

@Stranger6667
Copy link
Contributor

Before it led to an internal error

Ref: #5115

Not sure how to validate that the warning is emitted, tho :(

@codecov
Copy link

codecov bot commented Aug 10, 2019

Codecov Report

Merging #5729 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5729      +/-   ##
==========================================
+ Coverage   96.16%   96.17%   +<.01%     
==========================================
  Files         117      117              
  Lines       25865    25873       +8     
  Branches     2498     2498              
==========================================
+ Hits        24874    24884      +10     
+ Misses        690      689       -1     
+ Partials      301      300       -1
Impacted Files Coverage Δ
src/_pytest/config/__init__.py 94.64% <100%> (+0.01%) ⬆️
testing/test_warnings.py 100% <100%> (ø) ⬆️
testing/test_pastebin.py 100% <0%> (ø) ⬆️
src/_pytest/pastebin.py 91.22% <0%> (+3.5%) ⬆️

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 a295a3d...7183335. Read the comment docs.

Copy link
Contributor

@blueyed blueyed left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@blueyed blueyed left a comment

Choose a reason for hiding this comment

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

Thanks!

(please squash and force-push, #4361)

@Stranger6667
Copy link
Contributor Author

Thank you very much, Daniel! :)

@nicoddemus
Copy link
Member

@Stranger6667 hope you don't mind, I went ahead and squashed this myself given that you had agreed to squash before merging. 👍

@Stranger6667
Copy link
Contributor Author

Yes, sure, thank you @nicoddemus :)

@nicoddemus nicoddemus merged commit 0a62c4a into pytest-dev:master Aug 15, 2019
@Stranger6667 Stranger6667 deleted the issue-5115 branch August 15, 2019 11:07
@blueyed
Copy link
Contributor

blueyed commented Aug 15, 2019

@Stranger6667
Thanks for fixing my issue.. :)

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.

4 participants