Further comments to disable warnings on various lines

prospector/config/__init__.py

@@ -12,7 +12,9 @@ class ProspectorConfig(object):
     # There are several methods on this class which could technically
     # be functions (they don't use the 'self' argument) but that would
     # make this module/class a bit ugly.
-    # pylint:disable=no-self-use
+    # Also the 'too many instance attributes' warning is ignored, as this
+    # is a config object and its sole purpose is to hold many properties!
+    # pylint:disable=no-self-use,too-many-instance-attributes
 
     def __init__(self):
         self.config, self.arguments = self._configure_prospector()
@@ -29,6 +31,7 @@ def __init__(self):
         self.libraries = self._find_used_libraries(self.config)
         self.tools_to_run = self._determine_tool_runners(self.config, self.profile)
         self.ignores = self._determine_ignores(self.config, self.profile, self.libraries)
+        self.configured_by = {}
 
     def get_tools(self, found_files):
         self.configured_by = {}

prospector/tools/frosted/__init__.py

@@ -88,6 +88,7 @@ def run(self, found_files):
             try:
                 check_path(filepath, reporter)
             except UnicodeDecodeError:
+                # pylint:disable=pointless-except
                 pass
 
         return reporter.get_messages()

prospector/tools/pylint/__init__.py

@@ -28,7 +28,7 @@ def flush(self):
         pass
 
 
-class stdout_wrapper(object):
+class stdout_wrapper(object):  # noqa
 
     def __init__(self, hide_stdout):
         self.hide_stdout = hide_stdout
@@ -47,6 +47,10 @@ def __exit__(self, exc_type, exc_val, exc_tb):
 
 
 class PylintTool(ToolBase):
+    # There are several methods on this class which could technically
+    # be functions (they don't use the 'self' argument) but that would
+    # make this module/class a bit ugly.
+    # pylint:disable=no-self-use
 
     def __init__(self):
         self._args = self._extra_sys_path = None