Cleanup (#93)

* updating packages * cleaning up issues with upgraded pylint
jackdewinter · Aug 26, 2022 · aff981a · aff981a
1 parent d0ae393
commit aff981a
Show file tree

Hide file tree

Showing 7 changed files with 130 additions and 154 deletions.
diff --git a/Pipfile b/Pipfile
@@ -11,7 +11,7 @@ isort = "==5.10.1"
 flake8 = "==4.0.1"
 flake8-bugbear = "==22.8.23"
 flake8-bandit = "==3.0.0"
-pylint = "==3.0.0a4"
+pylint = "==3.0.0a5"
 pyroma = "==4.0"
 pytest = "==7.1.2"
 pytest-cov = "==3.0.0"

diff --git a/Pipfile.lock b/Pipfile.lock
diff --git a/project_summarizer/plugins/cobertura_plugin.py b/project_summarizer/plugins/cobertura_plugin.py
@@ -64,9 +64,8 @@ def get_output_path(self) -> str:
         """
         return self.__output_path
 
-    @classmethod
     def add_command_line_arguments(
-        cls, parser: argparse.ArgumentParser
+        self, parser: argparse.ArgumentParser
     ) -> Tuple[str, str]:
         """
         Add a command line argument to denote the file to scan.

diff --git a/project_summarizer/plugins/junit_plugin.py b/project_summarizer/plugins/junit_plugin.py
@@ -61,9 +61,8 @@ def get_output_path(self) -> str:
         """
         return self.__output_path
 
-    @classmethod
     def add_command_line_arguments(
-        cls, parser: argparse.ArgumentParser
+        self, parser: argparse.ArgumentParser
     ) -> Tuple[str, str]:
         """
         Add a command line argument to denote the file to scan.

diff --git a/publish/coverage.json b/publish/coverage.json
@@ -6,8 +6,8 @@
         "totalCovered": 244
     },
     "lineLevel": {
-        "totalMeasured": 833,
-        "totalCovered": 833
+        "totalMeasured": 831,
+        "totalCovered": 831
     }
 }
 
diff --git a/setup.cfg b/setup.cfg
@@ -12,7 +12,7 @@ max-complexity = 18
 select = B,C,E,F,W,T4
 
 [pylint]
-disable = C0301, C0330, W0511, duplicate-code
+disable = C0301, W0511, duplicate-code
 extension-pkg-whitelist=win32api
 
 [tool:pytest]

diff --git a/test/patch_builtin_open.py b/test/patch_builtin_open.py
@@ -45,7 +45,7 @@ def register_exception(self, exact_file_name, file_mode, exception_message=None)
         """
         self.exception_map[exact_file_name] = (file_mode, exception_message)
 
-    # pylint: disable=consider-using-with
+    # pylint: disable=consider-using-with, unspecified-encoding
     def my_open(self, *args, **kwargs):
         """
         Provide alternate handling of the "builtins.open" function.
@@ -79,4 +79,4 @@ def my_open(self, *args, **kwargs):
             self.patched_open = self.mock_patcher.start()
             self.patched_open.side_effect = self.my_open
 
-    # pylint: enable=consider-using-with
+    # pylint: enable=consider-using-with, unspecified-encoding