Skip to content

doc: remove costlysetup example #6544

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

Merged
merged 1 commit into from
Jan 23, 2020

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Jan 23, 2020

It is not included with docs, and
example/costlysetup/sub_a/test_quick.py::test_quick sleeps for 5s,
slowing down doctesting unnecessarily.

It is not included with docs, and
`example/costlysetup/sub_a/test_quick.py::test_quick` sleeps for 5s,
slowing down `doctesting` unnecessarily.
Copy link
Member

@nicoddemus nicoddemus left a comment

Choose a reason for hiding this comment

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

Perhaps we could reintroduce it, but it might not even be necessary in the end anyway given nobody missed it. 👍

@blueyed
Copy link
Contributor Author

blueyed commented Jan 23, 2020

@nicoddemus from what I can tell it was never in the toctree of the docs.

@blueyed blueyed merged commit 863bab5 into pytest-dev:master Jan 23, 2020
@blueyed blueyed deleted the doc-rm-costlysetup branch January 23, 2020 12:21
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