view tests/test-check-flake8.t @ 5779:96ed73c5c6ca

doc: remove .strip() for a docstring This shouldn't be necessary, and by stripping it (and removing the trailing newline) it causes issues with Mercurial's doc/gendoc.py's RST output, since there's no newline separating this string and the thing that comes after it. I believe that the `.strip()` has been in there since the beginning, but I have not found a reason for its existence. It's possible that this was required in older Mercurial versions, but is no longer required? Notably, the tests (which include an invocation of `hg help evolution`) still pass with this change. ### Alternatives considered - Making gendoc.py robust against this. This was rejected since there's no need for the .strip() as far as I can tell, and this is the only case I know of that would need such logic.
author Kyle Lippincott <spectral@google.com>
date Wed, 17 Feb 2021 12:00:45 -0800
parents 39bc93749027
children 89349d870f6b
line wrap: on
line source

#require test-repo flake8

Copied from Mercurial core (60ee2593a270)

  $ cd "`dirname "$TESTDIR"`"

run flake8 on all tracked files ending in .py or with a python shebang

  $ hg files -0 'set:(**.py or grep("^#!.*python")) - removed()' \
  > -X hgext3rd/evolve/thirdparty \
  > 2>/dev/null \
  > | xargs -0 "$PYTHON" -m flake8