fix deprecation warnings and other lints #435
Open
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.
I vendor
docopt.py
in some of my projects, and I seeDeprecationWarning: invalid escape sequence
when I runpytest
across the whole project, because it's searchingdocopt.py
for tests and hitting those warnings. As part of fixing that, I figured it would be nice to fix all theflake8
lints, at least in that one file. There are a few:#
.I also added
flake8
totox.ini
, which helped to catch that 2.7-only lint above.If this is all too aggressive, I could rip out all of these changes except the escape sequence fixes, which is the only thing I think we genuinely need.