Merge pull request #2643 from coolo/fix_e265

Remove E265 from flake ignore list

.flake8

@@ -1,4 +1,4 @@
 [flake8]
 exclude = abichecker
 max-line-length = 100
-ignore = E501,F401,E128,E251,E201,E202,E302,E305,F841,E261,E265,E712,E126,E711,E125,E123,E101,E124,E127,E701,E714,W504,E129,E741,E722,E731
+ignore = E501,F401,E128,E251,E201,E202,E302,E305,F841,E261,E712,E126,E711,E125,E123,E101,E124,E127,E701,E714,W504,E129,E741,E722,E731

biarchtool.py

@@ -143,7 +143,7 @@ class BiArchTool(ToolBase.ToolBase):
             self.packages = self.meta_get_packagelist(self.project)
         elif packages == '__latest__':
             # only works when called in packagelists loop
-            #self.packages = self._filter_packages_by_time(self.latest_packages(self.project))
+            # self.packages = self._filter_packages_by_time(self.latest_packages(self.project))
             self.packages = self.latest_packages(self.project)
         else:
             self.packages = packages

dashboard/generate.py

@@ -89,7 +89,6 @@ class Fetcher(object):
                 unresolvable += count
                 continue
             building += count
-            #print(code, file=sys.stderr)
         # let's count them as building
         if building > 0:
             building += unresolvable

factory-package-news/factory-package-news.py

@@ -222,7 +222,7 @@ class ChangeLogger(cmdln.Cmdln):
         details = ''
         for srpm in sorted(group.keys()):
             srpm1 = v1pkgs[group[srpm][0]]['sourcerpm']
-            #print group[srpm], srpm, srpm1
+            # print group[srpm], srpm, srpm1
             if srpm1 == srpm:
                 continue # source package unchanged
             try:

fcc_submitter.py

@@ -183,7 +183,7 @@ class FccSubmitter(object):
 
         f = osc.core.show_prj_results_meta(self.apiurl, project)
         root = ET.fromstringlist(f)
-        #print ET.dump(root)
+        # print ET.dump(root)
 
         failed_multibuild_pacs = []
         pacs = []

gocd/report-status.py

@@ -86,7 +86,7 @@ if __name__ == '__main__':
     args = parser.parse_args()
     # Configure OSC
     osc.conf.get_config(override_apiurl=args.apiurl)
-    #osc.conf.config['debug'] = 1
+    # osc.conf.config['debug'] = 1
 
     architectures = sorted(target_archs(args.apiurl, args.project, args.repository))
     for arch in architectures:

leaper.py

@@ -285,7 +285,7 @@ class Leaper(ReviewBot.ReviewBot):
 
             # TODO #1662: Uncomment once maintbot has been superseded and leaper
             # is no longer run in comment-only mode.
-            #self.do_check_maintainer_review = True
+            # self.do_check_maintainer_review = True
 
             return None
         elif self.action.type == 'maintenance_incident':

osclib/PubSubConsumer.py

@@ -378,7 +378,7 @@ class PubSubConsumer(object):
             self.logger.debug('Stopping')
             if self._consuming:
                 self.stop_consuming()
-                #self._connection.ioloop.start()
+                # self._connection.ioloop.start()
             else:
                 self._connection.ioloop.stop()
             self.logger.debug('Stopped')

osclib/cleanup_rings.py

@@ -8,7 +8,7 @@ from osclib.core import builddepinfo
 try:
     from urllib.error import HTTPError
 except ImportError:
-    #python 2.x
+    # python 2.x
     from urllib2 import HTTPError
 
 class CleanupRings(object):

sync-rebuild.py

@@ -15,7 +15,7 @@ architectures = ["x86_64", "i586"]
 pkg = ""
 projects = ['openSUSE:Factory', 'openSUSE:Factory:Rebuild']
 
-#initialize osc config
+# initialize osc config
 osc.conf.get_config()
 
 def get_prj_results(prj, arch):
@@ -61,7 +61,6 @@ def check_pkgs(rebuild_list):
 
 def rebuild_pkg_in_factory(package, prj, arch, testmode, code=None):
     query = { 'cmd': 'rebuild', 'arch': arch }
-    #prj = "home:jzwickl"
     if package:
         query['package'] = package
     pkg = query['package']

update_crawler.py

@@ -206,7 +206,7 @@ class UpdateCrawler(object):
                                 ['source', prj, '00Meta', 'lookup.yml']))
 
     def follow_link(self, project, package, rev, verifymd5):
-        #print "follow", project, package, rev
+        # print "follow", project, package, rev
         # verify it's still the same package
         xml = ET.fromstring(self._get_source_package(project, package, rev))
         if xml.get('verifymd5') != verifymd5:
@@ -272,7 +272,7 @@ class UpdateCrawler(object):
                 md5_from = sourceinfo.get('verifymd5')
                 md5_to = targetinfo.get('verifymd5')
                 if md5_from == md5_to:
-                    #logging.info('Package %s not marked for update' % package)
+                    # logging.info('Package %s not marked for update' % package)
                     continue
 
                 if self.is_source_innerlink(self.to_prj, package):