lint: remove old pylint comments

Author: elopez

crytic_compile/__main__.py

@@ -148,7 +148,7 @@ def parse_args() -> argparse.Namespace:
     return args
 
 
-class ShowPlatforms(argparse.Action):  # pylint: disable=too-few-public-methods
+class ShowPlatforms(argparse.Action):
     """
     This class is used to print the different platforms supported to the log
     See --supported-platforms

crytic_compile/compilation_unit.py

@@ -20,7 +20,6 @@
     from crytic_compile import CryticCompile
 
 
-# pylint: disable=too-many-instance-attributes
 class CompilationUnit:
     """CompilationUnit class"""
 
@@ -239,7 +238,6 @@ def filename_lookup(self, filename: str) -> Filename:
         Returns:
             Filename: Associated Filename object
         """
-        # pylint: disable=import-outside-toplevel
         from crytic_compile.platform.hardhat import Hardhat
         from crytic_compile.platform.truffle import Truffle
 

crytic_compile/compiler/compiler.py

@@ -7,7 +7,6 @@
 LOGGER = logging.getLogger("CryticCompile")
 
 
-# pylint: disable=too-few-public-methods
 class CompilerVersion:
     """
     Class representing the compiler information

crytic_compile/crytic_compile.py

@@ -41,9 +41,6 @@
 logging.basicConfig()
 
 
-# pylint: disable=too-many-lines
-
-
 def get_platforms() -> list[type[AbstractPlatform]]:
     """Return the available platforms classes in order of preference
 
@@ -111,13 +108,11 @@ def _configure_solc(solc_requested: str, offline: bool) -> str:
     return solc_path.absolute().as_posix()
 
 
-# pylint: disable=too-many-instance-attributes
 class CryticCompile:
     """
     Main class.
     """
 
-    # pylint: disable=too-many-branches
     def __init__(self, target: str | AbstractPlatform, **kwargs: str) -> None:
         """See https://github.com/crytic/crytic-compile/wiki/Configuration
         Target is usually a file or a project directory. It can be an AbstractPlatform
@@ -152,7 +147,6 @@ def __init__(self, target: str | AbstractPlatform, **kwargs: str) -> None:
         else:
             self._working_dir = Path.cwd()
 
-        # pylint: disable=too-many-nested-blocks
         if isinstance(target, str):
             platform = self._init_platform(target, **kwargs)
             # If the platform is Solc it means we are trying to compile a single
@@ -588,7 +582,6 @@ def export(self, **kwargs: str) -> list[str]:
     ###################################################################################
     ###################################################################################
 
-    # pylint: disable=no-self-use
     def _init_platform(self, target: str, **kwargs: str) -> AbstractPlatform:
         """Init the platform
 

crytic_compile/platform/abstract_platform.py

@@ -20,11 +20,9 @@ class IncorrectPlatformInitialization(Exception):
     Exception raises if a platform was not properly defined
     """
 
-    # pylint: disable=unnecessary-pass
     pass
 
 
-# pylint: disable=too-many-instance-attributes
 @dataclass
 class PlatformConfig:
     """
@@ -179,7 +177,7 @@ def is_dependency(self, path: str) -> bool:
         return False
 
     @staticmethod
-    def config(working_dir: str) -> PlatformConfig | None:  # pylint: disable=unused-argument
+    def config(working_dir: str) -> PlatformConfig | None:
         """Return configuration data that should be passed to solc, such as version, remappings ecc.
 
         Args:

crytic_compile/platform/all_platforms.py

@@ -1,4 +1,3 @@
 """
 Module containing all the platforms
 """
-# pylint: disable=unused-import

crytic_compile/platform/archive.py

@@ -75,7 +75,7 @@ def compile(self, crytic_compile: "CryticCompile", **_kwargs: str) -> None:
             crytic_compile (CryticCompile): associated CryticCompile object
             **_kwargs: unused
         """
-        # pylint: disable=import-outside-toplevel
+
         from crytic_compile.crytic_compile import get_platforms
 
         try:

crytic_compile/platform/brownie.py

@@ -79,7 +79,6 @@ def compile(self, crytic_compile: "CryticCompile", **kwargs: str) -> None:
                         LOGGER.error(stderr)
 
             except OSError as error:
-                # pylint: disable=raise-missing-from
                 raise InvalidCompilation(error)
 
         if not os.path.isdir(os.path.join(self._target, build_directory)):
@@ -135,7 +134,6 @@ def _guessed_tests(self) -> list[str]:
         return ["brownie test"]
 
 
-# pylint: disable=too-many-locals
 def _iterate_over_files(
     crytic_compile: "CryticCompile", target: Path, filenames: list[Path]
 ) -> None:

crytic_compile/platform/buidler.py

@@ -36,7 +36,6 @@ class Buidler(AbstractPlatform):
     PROJECT_URL = "https://github.com/nomiclabs/buidler"
     TYPE = Type.BUILDER
 
-    # pylint: disable=too-many-locals,too-many-statements,too-many-branches
     def compile(self, crytic_compile: "CryticCompile", **kwargs: str) -> None:
         """Run the compilation
 

crytic_compile/platform/dapp.py

@@ -39,7 +39,6 @@ class Dapp(AbstractPlatform):
     PROJECT_URL = "https://github.com/dapphub/dapptools"
     TYPE = Type.DAPP
 
-    # pylint: disable=too-many-locals
     def compile(self, crytic_compile: "CryticCompile", **kwargs: str) -> None:
         """Run the compilation
 
@@ -189,7 +188,7 @@ def _run_dapp(target: str) -> None:
     Raises:
         InvalidCompilation: If dapp failed to run
     """
-    # pylint: disable=import-outside-toplevel
+
     from crytic_compile.platform.exceptions import InvalidCompilation
 
     cmd = ["dapp", "build"]
@@ -204,7 +203,6 @@ def _run_dapp(target: str) -> None:
         ) as process:
             _, _ = process.communicate()
     except OSError as error:
-        # pylint: disable=raise-missing-from
         raise InvalidCompilation(error)