Adjust code style to be pep8 compatible

Author: psss

LICENSE

@@ -337,4 +337,3 @@ proprietary programs.  If your program is a subroutine library, you may
 consider it more useful to permit linking proprietary applications with the
 library.  If this is what you want to do, use the GNU Lesser General
 Public License instead of this License.
-

bin/did

@@ -34,8 +34,8 @@ range. By default all available stats for this week are reported.
 
 import sys
 
-import did.cli
 import did.base
+import did.cli
 import did.utils
 
 try:

did/base.py

@@ -2,23 +2,23 @@
 
 """ Config, Date, User and Exceptions """
 
+import codecs
+import configparser
+import datetime
 import io
 import os
 import re
 import sys
-import codecs
-import datetime
-import optparse
-import configparser
-from dateutil.relativedelta import MO as MONDAY, FR as FRIDAY
-from datetime import timedelta
 from configparser import NoOptionError, NoSectionError
+from datetime import timedelta
+
+from dateutil.relativedelta import FR as FRIDAY
+from dateutil.relativedelta import MO as MONDAY
 from dateutil.relativedelta import relativedelta as delta
 
 from did import utils
 from did.utils import log
 
-
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 #  Constants
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@@ -51,15 +51,19 @@
 class GeneralError(Exception):
     """ General stats error """
 
+
 class ConfigError(GeneralError):
     """ Stats configuration problem """
 
+
 class ConfigFileError(ConfigError):
     """ Problem with the config file """
 
+
 class OptionError(GeneralError):
     """ Invalid command line """
 
+
 class ReportError(GeneralError):
     """ Report generation error """
 
@@ -110,7 +114,7 @@ def plugins(self):
         """ Custom plugins """
         try:
             return self.parser.get("general", "plugins")
-        except:
+        except BaseException:
             return None
 
     @property

did/cli.py

@@ -7,9 +7,10 @@
 running the main loop which gathers all individual stats.
 """
 
+import argparse
 import re
 import sys
-import argparse
+
 from dateutil.relativedelta import relativedelta as delta
 
 import did.base

did/plugins/bodhi.py

@@ -11,17 +11,17 @@
 
 """
 
-from did.utils import log, pretty, listed
-from did.base import Config, ReportError
-from did.stats import Stats, StatsGroup
-
 from bodhi.client.bindings import BodhiClient
 
+from did.base import Config, ReportError
+from did.stats import Stats, StatsGroup
+from did.utils import listed, log, pretty
 
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 #  Investigator
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
+
 class Bodhi(object):
     """ Bodhi """
 
@@ -57,6 +57,7 @@ def search(self, query):
 
 class Update(object):
     """ Bodhi update """
+
     def __init__(self, data):
         self.data = data
         self.title = data['title']
@@ -76,12 +77,13 @@ def __str__(self):
 
 class UpdatesCreated(Stats):
     """ Updates created """
+
     def fetch(self):
         log.info('Searching for updates created by {0}'.format(self.user))
         self.stats = [
             Update(update) for update in self.parent.bodhi.search(
-            query='updates/?user={0}&submitted_before={1}'
-                  '&submitted_since={2}'.format(
+                query='updates/?user={0}&submitted_before={1}'
+                '&submitted_since={2}'.format(
                       self.user.login, self.options.until.date,
                       self.options.since.date))]
 

did/plugins/bugzilla.py

@@ -42,14 +42,14 @@
 .. _Preferences: https://bugzilla.redhat.com/userprefs.cgi?tab=apikey
 """
 
-import bugzilla
 import xmlrpc.client
 
+import bugzilla
+
 from did.base import Config, ReportError
 from did.stats import Stats, StatsGroup
 from did.utils import log, pretty, split
 
-
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 #  Constants
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@@ -274,13 +274,15 @@ def subscribed(self, user):
 #  Bugzilla Stats
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
+
 class VerifiedBugs(Stats):
     """
     Bugs verified
 
     Bugs with ``QA Contact`` field set to given user or changed by the
     given user and having their status changed to ``VERIFIED``.
     """
+
     def fetch(self):
         log.info("Searching for bugs verified by {0}".format(self.user))
         # Common query options
@@ -328,6 +330,7 @@ class ReturnedBugs(Stats):
     the ``ASSIGNED`` status, meaning the fix for the issue is not
     correct or complete.
     """
+
     def fetch(self):
         log.info("Searching for bugs returned by {0}".format(self.user))
         query = {
@@ -364,6 +367,7 @@ class FiledBugs(Stats):
 
     Newly created bugs by given user, marked as the ``Reporter``.
     """
+
     def fetch(self):
         log.info("Searching for bugs filed by {0}".format(self.user))
         query = {
@@ -465,6 +469,7 @@ class PostedBugs(Stats):
     Bugs with patches posted for review, detected by their status
     change to ``POST`` and given user set as ``Assignee``.
     """
+
     def fetch(self):
         log.info("Searching for bugs posted by {0}".format(self.user))
         query = {
@@ -499,6 +504,7 @@ class PatchedBugs(Stats):
     denoting the patch for the issue is available (e.g. attached
     to the bug or pushed to a feature git branch).
     """
+
     def fetch(self):
         log.info("Searching for bugs patched by {0}".format(self.user))
         query = {
@@ -563,6 +569,7 @@ class CommentedBugs(Stats):
 
     All bugs commented by given user in requested time frame.
     """
+
     def fetch(self):
         log.info("Searching for bugs commented by {0}".format(self.user))
         query = {
@@ -591,6 +598,7 @@ class SubscribedBugs(Stats):
 
     All bugs subscribed by given user in requested time frame.
     """
+
     def fetch(self):
         log.info("Searching for bugs subscribed by {0}".format(self.user))
         query = {

did/plugins/confluence.py

@@ -32,17 +32,18 @@
   higher priority.
 """
 
+import distutils.util
 import os
 import re
-import requests
 import urllib.parse
-import distutils.util
-from requests_gssapi import HTTPSPNEGOAuth, DISABLED
+
+import requests
 from requests.packages.urllib3.exceptions import InsecureRequestWarning
+from requests_gssapi import DISABLED, HTTPSPNEGOAuth
 
-from did.utils import log, pretty, listed
 from did.base import Config, ReportError
 from did.stats import Stats, StatsGroup
+from did.utils import listed, log, pretty
 
 # Maximum number of results fetched at once
 MAX_RESULTS = 100
@@ -234,7 +235,7 @@ def __init__(self, option, name=None, parent=None, user=None):
                 option=option + "-comments",
                 parent=self,
                 name="Comments added in {}".format(option)),
-        ]
+            ]
 
     @property
     def session(self):

did/plugins/footer.py

@@ -13,11 +13,11 @@
 
 from did.stats import EmptyStatsGroup
 
-
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 #  Footer
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
+
 class Footer(EmptyStatsGroup):
     """ Footer """
     # Show footer at the bottom

did/plugins/gerrit.py

@@ -14,19 +14,19 @@
 """
 
 import json
-import urllib.request
 import urllib.parse
+import urllib.request
 from datetime import datetime
 
-from did.utils import log, pretty
+from did.base import TODAY, Config, ReportError
 from did.stats import Stats, StatsGroup
-from did.base import Config, ReportError, TODAY
-
+from did.utils import log, pretty
 
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 #  Change
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
+
 class Change(object):
     """ Request gerrit change """
 
@@ -57,10 +57,12 @@ def __hash__(self):
 #  Gerrit Stats
 # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
+
 class Gerrit(object):
     """
      curl -s 'https://REPOURL/gerrit/changes/?q=is:abandoned+age:7d'
     """
+
     def __init__(self, baseurl, prefix):
         self.opener = urllib.request.FancyURLopener()
         self.baseurl = baseurl
@@ -122,6 +124,7 @@ class GerritUnit(Stats):
         General mother class offering general services for querying
         Gerrit repo.
     """
+
     def __init__(
             self, option, name=None, parent=None, base_url=None, prefix=None):
         self.base_url = base_url if base_url is not None else parent.repo_url
@@ -209,6 +212,7 @@ class AbandonedChanges(GerritUnit):
     """
     Changes abandoned
     """
+
     def fetch(self):
         log.info("Searching for changes abandoned by {0}".format(self.user))
         self.stats = GerritUnit.fetch(self, 'status:abandoned')
@@ -220,6 +224,7 @@ class MergedChanges(GerritUnit):
     """
     Changes successfully merged
     """
+
     def fetch(self):
         log.info("Searching for changes merged by {0}".format(self.user))
         self.stats = GerritUnit.fetch(self, 'status:merged')
@@ -238,35 +243,40 @@ class SubmitedChanges(GerritUnit):
     """
     Changes submitted for review
     """
+
     def fetch(self):
         log.info("Searching for changes opened by {0}".format(self.user))
         if 'wip' in self.server_features:
             query_string = 'status:open -is:wip'
         else:
             query_string = 'status:open'
         self.stats = GerritUnit.fetch(self, query_string,
-            limit_since=True)
+                                      limit_since=True)
         log.debug("self.stats = {0}".format(self.stats))
 
+
 class WIPChanges(GerritUnit):
     """
     Work in progress changes
     """
+
     def fetch(self):
         log.info("Searching for WIP changes opened by {0}".format(self.user))
         if 'wip' not in self.server_features:
             log.debug("WIP reviews are not supported by this server")
             return []
         self.stats = GerritUnit.fetch(self, 'status:open is:wip',
-            limit_since=True)
+                                      limit_since=True)
         log.debug("self.stats = {0}".format(self.stats))
 
+
 class AddedPatches(GerritUnit):
     # curl -s 'https://REPOURL\
     #    /changes/?q=is:closed+owner:mcepl&q=is:open+owner:mcepl
     """
     Additional patches added to existing changes
     """
+
     def fetch(self):
         log.info("Searching for patches added to changes by {0}".format(
             self.user))
@@ -320,6 +330,7 @@ class ReviewedChanges(GerritUnit):
     """
     Review of a change (for reviewers)
     """
+
     def fetch(self):
         log.info("Searching for changes reviewed by {0}".format(self.user))
         # Collect ALL changes opened (and perhaps now closed) after
@@ -402,7 +413,7 @@ def __init__(self, option, name=None, parent=None, user=None):
                 "No prefix set in the [{0}] section".format(option))
 
         self.server_features = []
-        if self.config.get('wip', True) == True:
+        if self.config.get('wip', True):
             self.server_features.append('wip')
 
         self.stats = [