Merge master into features and fix flake8 errors #2594
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Emit it only once per each generator, rather than for each generated function. Also add information about which test caused it to be emitted.
It is important enough that it should be easier to find
Add a link to our backwards compatibility policy to our side-bar
Make YIELD_TEST warning less spammy
…-docs Add docs for Item.add_report_section in the docs
We sometimes would see the following line: collected 1 item s just because previous write to the terminal includes number of characters greater than 'collected 1 item'.
Ensure final collected line doesn't include artifacts of previous write
W191 indentation contains tabs W292 no newline at end of file W293 blank line contains whitespace W391 blank line at end of file
indentation contains mixed spaces and tabs
indentation is not a multiple of four
unexpected indentation
continuation line under-indented for hanging indent
continuation line missing indentation or outdented
closing bracket does not match indentation of opening bracket’s line
closing bracket does not match visual indentation
continuation line with same indent as next logical line
continuation line over-indented for hanging indent
continuation line over-indented for visual indent
continuation line under-indented for visual indent
visually indented line with same indent as next logical line
continuation line unaligned for hanging indent
whitespace after ‘(‘
whitespace before ‘)’
whitespace before ‘:’
multiple spaces before operator
expected 2 blank lines, found 0
too many blank lines (3)
multiple imports on one line
module level import not at top of file
line too long (> 120 characters)
multiple statements on one line (colon)
multiple statements on one line (semicolon)
multiple statements on one line (def)
comparison to True should be ‘if cond is True:’ or ‘if cond:’
do not assign a lambda expression, use a def
pytest-dev#2574 Options --fixtures and --fixtures-per-test keep indentation of docstrings
The file apparently contains an early design document to what has become @pytest.fixture and can be deleted
Remove _pytest/impl file
…s-for-testing-plugin-code Extend pytester docs for testing plugin code
RonnyPfannschmidt
approved these changes
Jul 20, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Besides merging with master, also fix the flake8 errors on it as discussed in #2582; I will open a separate PR to add the tox env to run autopep8 and update docs.