Skip to content
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

Doc Fix: skipif's example description #3611

Merged
merged 6 commits into from Jun 23, 2018
Merged
Diff settings

Always

Just for now

Copy path View file
@@ -0,0 +1 @@
Minor Doc Fix: The description above the example for ``@pytest.fixture.skipif`` now matches the code
Copy path View file
@@ -80,7 +80,7 @@ during import time.

If you wish to skip something conditionally then you can use ``skipif`` instead.
Here is an example of marking a test function to be skipped
when run on a Python3.6 interpreter::
when run on an interpreter earlier than Python3.6 ::

import sys
@pytest.mark.skipif(sys.version_info < (3,6),

This comment has been minimized.

Copy link
@AdamEr8

AdamEr8 Jun 22, 2018

Author Contributor

Maybe the reason "requires python3.6" should also change to "requires python3.6 or higher" to fully match the condition

This comment has been minimized.

Copy link
@nicoddemus

nicoddemus Jun 22, 2018

Member

Agreed!

ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.