Skip to content

syntax: Fix Python 3.6 string escape deprecation - #389

Merged
mihaic merged 1 commit into
brainiak:masterfrom
mihaic:fix-w605
Oct 24, 2018
Merged

syntax: Fix Python 3.6 string escape deprecation#389
mihaic merged 1 commit into
brainiak:masterfrom
mihaic:fix-w605

Conversation

@mihaic

@mihaic mihaic commented Oct 24, 2018

Copy link
Copy Markdown
Member

From Python 3.6 release notes:
"A backslash-character pair that is not a valid escape sequence now
generates a DeprecationWarning. Although this will eventually become a
SyntaxError, that will not be for several Python releases."
https://docs.python.org/3/whatsnew/3.6.html#deprecated-python-behavior

This became a run-checks.sh error and was causing builds to fail in PR #387 starting with pycodestyle 2.4.0:
https://pycodestyle.readthedocs.io/en/latest/developer.html

From Python 3.6 release notes:
"A backslash-character pair that is not a valid escape sequence now
generates a DeprecationWarning. Although this will eventually become a
SyntaxError, that will not be for several Python releases."
https://docs.python.org/3/whatsnew/3.6.html#deprecated-python-behavior

This became a run-checks.sh error and was causing builds to fail in PR brainiak#387 starting with pycodestyle 2.4.0:
https://pycodestyle.readthedocs.io/en/latest/developer.html
@mihaic
mihaic requested a review from lcnature October 24, 2018 18:32

@lcnature lcnature left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since all the changes are related to backslashes, this looks good.

@mihaic

mihaic commented Oct 24, 2018

Copy link
Copy Markdown
Member Author

FYI, this is why the Conda builds are failing on Travis:
matplotlib/matplotlib#12601

As far as I can see on the Matplotlib email list, they will only release a version incorporating the fix next weekend, so I am making the test non-blocking until then.

@mihaic
mihaic merged commit 224d2ee into brainiak:master Oct 24, 2018
@mihaic
mihaic deleted the fix-w605 branch October 24, 2018 23:52
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