-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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 into features #5744
Conversation
Improve docs of pytest.importorskip
Co-Authored-By: Bruno Oliveira <[email protected]>
Update URL: python/black → psf/black
Likely to fix flaky coverage due to requests failing sometimes. Ref: https://codecov.io/gh/pytest-dev/pytest/compare/f7e81dab9aa8d57498511e1b5655b53cbfbed0d0...83a1f4bd668fe337d42f909cc2ef2d8d986e9748/changes
remove %s formatting from docs
test_non_ascii_paste_text: mock call to urlopen
Co-Authored-By: Bruno Oliveira <[email protected]>
Do not treat warnings as errors during ``pytest_configure``.
warn for async generator functions
…-dev#5701) check that tests that are partial staticmethods are supported
Async result warn
Conflicts: src/_pytest/outcomes.py
Codecov Report
@@ Coverage Diff @@
## features #5744 +/- ##
============================================
+ Coverage 96.25% 96.25% +<.01%
============================================
Files 117 117
Lines 25902 25948 +46
Branches 2495 2497 +2
============================================
+ Hits 24932 24977 +45
- Misses 666 667 +1
Partials 304 304
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Handling dropped coverage in #5747.
@@ -251,7 +257,7 @@ def get_real_method(obj, holder): | |||
try: | |||
is_method = hasattr(obj, "__func__") | |||
obj = get_real_func(obj) | |||
except Exception: | |||
except Exception: # pragma: no cover |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wut?
Rather leave it show up in reports..
Thanks! Please rebase #5747 on |
You can/should release regardless of #5747, no? (which is not going to merged right away) |
Yep, only later I saw it closely and noticed it did not affect the release in the end. 👍 |
No description provided.