launchpad-reviewers team mailing list archive

[Ioana Lasc <ioana.lasc@xxxxxxxxxxxxx>]

Ioana Lasc has proposed merging ~ilasc/launchpad:flake8-e502 into launchpad:master.

Commit message:
flake8: fix E502 backslash redundant between brackets

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ilasc/launchpad/+git/launchpad/+merge/406630
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilasc/launchpad:flake8-e502 into launchpad:master.

diff --git a/lib/lp/bugs/browser/bugtask.py b/lib/lp/bugs/browser/bugtask.py
index 48085a8..5769a3f 100644
--- a/lib/lp/bugs/browser/bugtask.py
+++ b/lib/lp/bugs/browser/bugtask.py
@@ -254,9 +254,9 @@ def get_comments_for_bugtask(bugtask, truncate=False, for_display=False,
     comments = sorted(comments.values(), key=attrgetter("index"))
     current_title = bugtask.bug.title
     for comment in comments:
-        if not ((unique_title(comment.title) == \
-                 unique_title(current_title)) or \
-                (unique_title(comment.title) == \
+        if not ((unique_title(comment.title) ==
+                 unique_title(current_title)) or
+                (unique_title(comment.title) ==
                  unique_title(bugtask.bug.title))):
             # this comment has a new title, so make that the rolling focus
             current_title = comment.title
diff --git a/lib/lp/bugs/browser/tests/test_bugsubscription_views.py b/lib/lp/bugs/browser/tests/test_bugsubscription_views.py
index 5dd87d0..85fc62b 100644
--- a/lib/lp/bugs/browser/tests/test_bugsubscription_views.py
+++ b/lib/lp/bugs/browser/tests/test_bugsubscription_views.py
@@ -629,7 +629,7 @@ class BugPortletSubscribersWithDetailsTests(TestCaseWithFactory):
                 'can_edit': False,
                 'web_link': canonical_url(subscriber),
                 'self_link': absoluteURL(subscriber, api_request),
-                'display_subscribed_by': \
+                'display_subscribed_by':
                     'Subscribed by Team Owner (team-owner)',
                 },
             'subscription_level': "Lifecycle",
@@ -682,7 +682,7 @@ class BugPortletSubscribersWithDetailsTests(TestCaseWithFactory):
                     'web_link': canonical_url(naked_subscriber,
                         rootsite='mainsite'),
                     'self_link': absoluteURL(naked_subscriber, api_request),
-                    'display_subscribed_by': \
+                    'display_subscribed_by':
                         'Subscribed by Team Owner (team-owner)',
                     },
                 'subscription_level': "Lifecycle",
@@ -734,7 +734,7 @@ class BugPortletSubscribersWithDetailsTests(TestCaseWithFactory):
                 'can_edit': True,
                 'web_link': canonical_url(subscriber),
                 'self_link': absoluteURL(subscriber, api_request),
-                'display_subscribed_by': \
+                'display_subscribed_by':
                     'Subscribed by Team Owner (team-owner)',
                 },
             'subscription_level': "Lifecycle",
@@ -768,7 +768,7 @@ class BugPortletSubscribersWithDetailsTests(TestCaseWithFactory):
                 'can_edit': True,
                 'web_link': canonical_url(subscriber),
                 'self_link': absoluteURL(subscriber, api_request),
-                'display_subscribed_by': \
+                'display_subscribed_by':
                     'Subscribed by Team Owner (team-owner)',
                 },
             'subscription_level': "Lifecycle",
diff --git a/lib/lp/services/fields/__init__.py b/lib/lp/services/fields/__init__.py
index 5281d57..87806e4 100644
--- a/lib/lp/services/fields/__init__.py
+++ b/lib/lp/services/fields/__init__.py
@@ -921,8 +921,8 @@ class WorkItemsText(Text):
 
         if milestone is None:
             raise LaunchpadValidationError("The milestone '%s' is not valid "
-                                           "for the target '%s'." % \
-                                               (milestone_name, target.name))
+                                           "for the target '%s'." %
+                                           (milestone_name, target.name))
         return milestone