Pylint has been updated => less false-negative

Author: penguinolog

exec_helpers/_ssh_base.py

@@ -367,7 +367,6 @@ def __build_connection_chain(self) -> _SSHConnChainT:
         conn_chain.append((config, auth))
 
         while config.proxyjump is not None:
-            # pylint: disable=no-member
             config = self.ssh_config[config.proxyjump]
             default_auth = ssh_auth.SSHAuth(username=config.user, key_filename=config.identityfile)
             conn_chain.append((config, self.__auth_mapping.get(config.hostname, default_auth)))

exec_helpers/_ssh_helpers.py

@@ -412,7 +412,6 @@ def _parse_paramiko_ssh_config(conf: paramiko.SSHConfig, host: str) -> HostsSSHC
     :return: parsed dictionary with proxy jump path, if available
     :rtype: HostsSSHConfigs
     """
-    # pylint: disable=no-member,unsubscriptable-object,unsupported-assignment-operation
     config = HostsSSHConfigs({host: SSHConfig.from_ssh_config(conf.lookup(host))})
     config.setdefault(config[host].hostname, config[host])
 
@@ -434,7 +433,6 @@ def _parse_dict_ssh_config(conf: SSHConfigsDictT, host: str) -> HostsSSHConfigs:
     :return: parsed dictionary with proxy jump path, if available
     :rtype: HostsSSHConfigs
     """
-    # pylint: disable=no-member,unsubscriptable-object,unsupported-assignment-operation
     config = HostsSSHConfigs({host: SSHConfig.from_ssh_config(conf.get(host, {"hostname": host}))})
     config.setdefault(config[host].hostname, config[host])
 
@@ -488,7 +486,6 @@ def parse_ssh_config(
         config = HostsSSHConfigs({host: SSHConfig(host)})
 
     if user_ssh_config is not None:
-        # pylint: disable=no-member,unsubscriptable-object,unsupported-assignment-operation
         user_config = _parse_paramiko_ssh_config(user_ssh_config, host)
         for hostname, cfg in user_config.items():
             config[hostname] = config[hostname].overridden_by(cfg)

exec_helpers/api.py

@@ -234,7 +234,7 @@ def __enter__(self) -> ExecHelper:
 
         .. versionchanged:: 1.1.0 lock on enter
         """
-        self.lock.acquire()
+        self.lock.acquire()  # pylint: disable=consider-using-with
         return self
 
     def __exit__(self, exc_type: typing.Any, exc_val: typing.Any, exc_tb: typing.Any) -> None:

exec_helpers/ssh_auth.py

@@ -365,7 +365,7 @@ def __setitem__(self, hostname: str, auth: SSHAuth) -> None:
             raise TypeError(f"Hostname should be string only! Got: {hostname!r}")
         if not isinstance(auth, SSHAuth):  # pragma: no cover
             raise TypeError(f"Value {auth!r} is not SSHAuth object!")
-        super().__setitem__(hostname.lower(), auth)  # pylint: disable=no-member
+        super().__setitem__(hostname.lower(), auth)
 
     def __getitem__(self, hostname: str) -> SSHAuth:
         """Dict-like access.
@@ -378,7 +378,7 @@ def __getitem__(self, hostname: str) -> SSHAuth:
         """
         if not isinstance(hostname, str):  # pragma: no cover
             raise TypeError(f"Hostname should be string only! Got: {hostname!r}")
-        return super().__getitem__(hostname.lower())  # pylint: disable=no-member
+        return super().__getitem__(hostname.lower())
 
     @typing.overload
     def get_with_alt_hostname(
@@ -436,4 +436,4 @@ def __delitem__(self, hostname: str) -> None:
         """
         if not isinstance(hostname, str):  # pragma: no cover
             raise TypeError(f"Hostname should be string only! Got: {hostname!r}")
-        super().__delitem__(hostname.lower())  # pylint: disable=no-member
+        super().__delitem__(hostname.lower())