[riot-notifications] [RIOT-OS/RIOT] tests/shell: fix flake8 errors in Python script (#15373)

Martine Lenders notifications at github.com
Thu Nov 5 10:57:57 CET 2020


@miri64 commented on this pull request.



> @@ -74,7 +74,7 @@
     ('echo escaped\\ space', '"echo""escaped space"'),
     ('echo escape within \'\\s\\i\\n\\g\\l\\e\\q\\u\\o\\t\\e\'', '"echo""escape""within""singlequote"'),
     ('echo escape within "\\d\\o\\u\\b\\l\\e\\q\\u\\o\\t\\e"', '"echo""escape""within""doublequote"'),
-    ("""echo "t\e st" "\\"" '\\'' a\ b""", '"echo""te st"""""\'""a b"'),
+    ("""echo "t\e st" "\\"" '\\'' a\ b""", '"echo""te st"""""\'""a b"'),  # noqa: W605

```suggestion
    ("echo \"t\e st\" \"\\"\" '\\'' a\ b", '"echo""te st"""""\'""a b"'),
```
... not sure this is easier to read. So maybe suppressing the warning is the better way to go :sweat_smile: 

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15373#pullrequestreview-524096821
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201105/3ee8b3cd/attachment.htm>


More information about the notifications mailing list