launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #27359
[Merge] ~cjwatson/launchpad:flake8-w605 into launchpad:master
Colin Watson has proposed merging ~cjwatson/launchpad:flake8-w605 into launchpad:master.
Commit message:
flake8: Fix another case of W605 invalid escape sequence
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/406646
I missed one of these in my previous commit.
--
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:flake8-w605 into launchpad:master.
diff --git a/utilities/format-imports b/utilities/format-imports
index 5e93624..df7e9ca 100755
--- a/utilities/format-imports
+++ b/utilities/format-imports
@@ -167,7 +167,7 @@ module_docstring_regex = re.compile(
# comments which are followed by an import line. Sometimes we even find
# lines that contain a single ")"... :-(
imports_section_regex = re.compile(
- "(^#.+\n)*^(import|(from ((?!__future__)\S+) import)).*\n"
+ "(^#.+\n)*^(import|(from ((?!__future__)\\S+) import)).*\n"
"(^import .+\n|^from .+\n|^[\t ]+.+\n|(^#.+\n)+((^import|^from) "
".+\n)|^\n|^[)]\n)*",
re.M)