Target E129 - Visually indented line with same indent

https://www.flake8rules.com/rules/E129.html

.flake8

@@ -1,4 +1,4 @@
 [flake8]
 exclude = abichecker
 max-line-length = 100
-ignore = W503,W504,E501,F401,E128,E251,E201,E202,E302,E305,F841,E261,E712,E126,E711,E125,E124,E714,E129
+ignore = W503,W504,E501,F401,E128,E251,E201,E202,E302,E305,F841,E261,E712,E126,E711,E125,E124,E714

ReviewBot.py

@@ -337,7 +337,7 @@ class ReviewBot(object):
                 r.by_package == by_package and
                 r.by_user == by_user and
                 # Only duplicate when allow_duplicate and state != new.
-                (not allow_duplicate or r.state == 'new')):
+                    (not allow_duplicate or r.state == 'new')):
                 del query['cmd']
                 self.logger.debug('skipped adding duplicate review for {}'.format(
                     '/'.join(query.values())))
@@ -445,7 +445,7 @@ class ReviewBot(object):
             # order from lowest to highest is: False, None, True.
             if overall is not False:
                 if ((overall is True and ret is not True) or
-                    (overall is None and ret is False)):
+                        (overall is None and ret is False)):
                     overall = ret
 
             if self.multiple_actions and ret is not None:

osclib/cleanup_rings.py

@@ -176,7 +176,7 @@ class CleanupRings(object):
         for source in self.sources:
             if (source not in self.pkgdeps and
                 source not in self.links and
-                source not in self.whitelist):
+                    source not in self.whitelist):
                 if source.startswith('texlive-specs-'): # XXX: texlive bullshit packaging
                     continue
                 # Expensive check so left until last.

osclib/core.py

@@ -955,7 +955,7 @@ def request_action_list_maintenance_incident(apiurl, project, package, states=['
             if action.type == 'maintenance_incident' and action.tgt_releaseproject == project and (
                 (action.tgt_package is None and
                     (action.src_package == package or action.src_package == package_repository)) or
-                (action.tgt_package == package_repository)):
+                    (action.tgt_package == package_repository)):
                 yield request, action
                 break
 
@@ -981,7 +981,7 @@ def request_action_list_maintenance_release(apiurl, project, package, states=['n
 
         for action in request.actions:
             if (action.type == 'maintenance_release' and
-                action.tgt_project == project and action.src_package == package_repository):
+                    action.tgt_project == project and action.src_package == package_repository):
                 yield request, action
                 break
 

osclib/origin.py

@@ -871,7 +871,7 @@ class devel_project_simulate:
     def get(self, apiurl, target_project, target_package):
         if (apiurl == self.apiurl and
             target_project == self.target_project and
-            target_package == self.target_package):
+                target_package == self.target_package):
             return self.devel_project, self.devel_package
 
         return False, False

osclib/request_splitter.py

@@ -176,7 +176,7 @@ class RequestSplitter(object):
         staging = self.stagings[staging]
         if (not staging['bootstrapped'] and
             staging['splitter_info']['strategy']['name'] in ('devel', 'super') and
-            staging['status'].get('state') not in ('acceptable', 'review')):
+                staging['status'].get('state') not in ('acceptable', 'review')):
             # Simplistic attempt to allow for followup requests to be staged
             # after age max has been passed while still stopping when ready.
             return True

osclib/stagingapi.py

@@ -470,7 +470,7 @@ class StagingAPI(object):
                 replace_old = request_old.find('state').get('name') in ['revoked', 'superseded', 'declined']
 
                 if (request_new.find('action').get('type') == 'delete' and
-                    request_old.find('action').get('type') == 'delete'):
+                        request_old.find('action').get('type') == 'delete'):
                     # Both delete requests.
                     if replace_old:
                         # Pointless since identical requests, but user desires.
@@ -484,7 +484,7 @@ class StagingAPI(object):
                         return stage_info, True
 
                 if (request_new.find('action').get('type') !=
-                    request_old.find('action').get('type')):
+                        request_old.find('action').get('type')):
                     # One delete and one submit.
                     if replace_old:
                         if self.ring_packages.get(target_package):