Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

Flake8 fixes #606

Closed
wants to merge 2 commits into from
Closed

Flake8 fixes #606

wants to merge 2 commits into from

Conversation

jkoelker
Copy link
Contributor

@jkoelker jkoelker commented Dec 5, 2018

Selectively uses raw strings to prevent escaping issues. Broken into two commits such that tests pass with each commit. Should fix the buildstatus.

Fixes the flake8 W605 for the tests:

test_marathon_lb.py:1175:-359: W605 invalid escape sequence '\ '
test_marathon_lb.py:1922:-110: W605 invalid escape sequence '\ '
test_marathon_lb.py:1922:-90: W605 invalid escape sequence '\ '
test_marathon_lb.py:2835:22: W605 invalid escape sequence '\ '
test_marathon_lb.py:2852:-81: W605 invalid escape sequence '\ '
test_marathon_lb.py:2931:-182: W605 invalid escape sequence '\ '
test_marathon_lb.py:2931:-157: W605 invalid escape sequence '\ '
Fixes the flake8 E605 for the lines:

./config.py:264:34: W605 invalid escape sequence '\ '
./config.py:592:61: W605 invalid escape sequence '\ '
./config.py:596:-84: W605 invalid escape sequence '\ '
@mesosphere-ci
Copy link

Can one of the admins verify this patch?

1 similar comment
@mesosphere-ci
Copy link

Can one of the admins verify this patch?

@jkoelker
Copy link
Contributor Author

jkoelker commented Dec 6, 2018

retest this please

@jkoelker
Copy link
Contributor Author

Closing in favor of #609.

@jkoelker jkoelker closed this Dec 20, 2018
@jkoelker jkoelker deleted the flake8 branch December 20, 2018 00:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants