-
-
Notifications
You must be signed in to change notification settings - Fork 18.2k
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
CI: autouse add_doctest_imports #57122
CI: autouse add_doctest_imports #57122
Conversation
@mroeschke - this gets doctests back to green, not noticing an appreciable slowdown. |
# ---------------------------------------------------------------- | ||
# Autouse fixtures | ||
# ---------------------------------------------------------------- | ||
@pytest.fixture(autouse=True) |
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.
Could you add a reference to the pytest issue you opened? (Still hoping we don't have to use an autouse
fixture one day)
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.
Done
Thanks @rhshadrach |
…57135) Backport PR #57122: CI: autouse add_doctest_imports Co-authored-by: Richard Shadrach <[email protected]>
* CI: autouse add_doctest_imports * add ref
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.Just trying this out to see impact to the CI
Ref: pytest-dev/pytest#11873
Effectively reverts #45667