Update existing suppression comments

Author: Jackenmen

redbot/cogs/audio/core/utilities/miscellaneous.py

@@ -378,9 +378,7 @@ def decode_track(self, track: str, decode_errors: str = "ignore") -> MutableMapp
         reader = DataReader(track)
 
         flags = (reader.read_int() & 0xC0000000) >> 30
-        (version,) = (
-            struct.unpack("B", reader.read_byte()) if flags & 1 != 0 else 1
-        )  # pylint: disable=unused-variable
+        (version,) = struct.unpack("B", reader.read_byte()) if flags & 1 != 0 else 1
 
         title = reader.read_utf().decode(errors=decode_errors)
         author = reader.read_utf().decode()

redbot/cogs/audio/manager.py

@@ -256,7 +256,7 @@ class ServerManager:
     def __init__(self, config: Config, cog: "Audio", timeout: Optional[int] = None) -> None:
         self.ready: asyncio.Event = asyncio.Event()
         self._config = config
-        self._proc: Optional[asyncio.subprocess.Process] = None  # pylint:disable=no-member
+        self._proc: Optional[asyncio.subprocess.Process] = None
         self._shutdown: bool = False
         self.start_monitor_task = None
         self.timeout = timeout
@@ -326,13 +326,11 @@ async def _start(self, java_path: str) -> None:
                 )
             )
         try:
-            self._proc = (
-                await asyncio.subprocess.create_subprocess_exec(  # pylint:disable=no-member
-                    *args,
-                    cwd=str(LAVALINK_DOWNLOAD_DIR),
-                    stdout=asyncio.subprocess.PIPE,
-                    stderr=asyncio.subprocess.STDOUT,
-                )
+            self._proc = await asyncio.subprocess.create_subprocess_exec(
+                *args,
+                cwd=str(LAVALINK_DOWNLOAD_DIR),
+                stdout=asyncio.subprocess.PIPE,
+                stderr=asyncio.subprocess.STDOUT,
             )
             log.info("Managed Lavalink node started. PID: %s", self._proc.pid)
             try:
@@ -413,13 +411,11 @@ async def _has_java(self) -> Tuple[bool, Optional[Tuple[int, int]]]:
 
     async def _get_java_version(self) -> Tuple[int, int]:
         """This assumes we've already checked that java exists."""
-        _proc: asyncio.subprocess.Process = (
-            await asyncio.create_subprocess_exec(  # pylint:disable=no-member
-                self._java_exc,
-                "-version",
-                stdout=asyncio.subprocess.PIPE,
-                stderr=asyncio.subprocess.PIPE,
-            )
+        _proc: asyncio.subprocess.Process = await asyncio.create_subprocess_exec(
+            self._java_exc,
+            "-version",
+            stdout=asyncio.subprocess.PIPE,
+            stderr=asyncio.subprocess.PIPE,
         )
         # java -version outputs to stderr
         _, err = await _proc.communicate()
@@ -533,7 +529,7 @@ async def _is_up_to_date(self):
             return True
         args, _ = await self._get_jar_args()
         args.append("--version")
-        _proc = await asyncio.subprocess.create_subprocess_exec(  # pylint:disable=no-member
+        _proc = await asyncio.subprocess.create_subprocess_exec(
             *args,
             cwd=str(LAVALINK_DOWNLOAD_DIR),
             stdout=asyncio.subprocess.PIPE,

redbot/cogs/downloader/repo_manager.py

@@ -53,9 +53,7 @@ class Candidate(NamedTuple):
     description: str
 
 
-class _RepoCheckoutCtxManager(
-    Awaitable[None], AsyncContextManager[None]
-):  # pylint: disable=duplicate-bases
+class _RepoCheckoutCtxManager(Awaitable[None], AsyncContextManager[None]):
     def __init__(
         self,
         repo: Repo,

redbot/cogs/economy/economy.py

@@ -601,9 +601,7 @@ async def slot_machine(author, channel, bid):
             sign = "  "
             if i == 1:
                 sign = ">"
-            slot += "{}{} {} {}\n".format(
-                sign, *[c.value for c in row]  # pylint: disable=no-member
-            )
+            slot += "{}{} {} {}\n".format(sign, *[c.value for c in row])
 
         payout = PAYOUTS.get(rows[1])
         if not payout:

redbot/core/bot.py

@@ -91,9 +91,7 @@ class _NoOwnerSet(RuntimeError):
 # d.py autoshardedbot should be at the end
 # all of our mixins should happen before,
 # and must include a call to super().__init__ unless they do not provide an init
-class Red(
-    commands.GroupMixin, RPCMixin, dpy_commands.bot.AutoShardedBot
-):  # pylint: disable=no-member # barely spurious warning caused by shadowing
+class Red(commands.GroupMixin, RPCMixin, dpy_commands.bot.AutoShardedBot):
     """Our subclass of discord.ext.commands.AutoShardedBot"""
 
     def __init__(self, *args, cli_flags=None, bot_dir: Path = Path.cwd(), **kwargs):

redbot/core/commands/requires.py

@@ -458,8 +458,8 @@ def reset(self) -> None:
         This will clear all rules, including defaults. It also resets
         the `Requires.ready_event`.
         """
-        self._guild_rules.clear()  # pylint: disable=no-member
-        self._global_rules.clear()  # pylint: disable=no-member
+        self._guild_rules.clear()
+        self._global_rules.clear()
         self.ready_event.clear()
 
     async def verify(self, ctx: "Context") -> bool:
@@ -922,12 +922,12 @@ class _IntKeyDict(Dict[int, _T]):
     def __getitem__(self, key: Any) -> _T:
         if not isinstance(key, int):
             raise TypeError("Keys must be of type `int`")
-        return super().__getitem__(key)  # pylint: disable=no-member
+        return super().__getitem__(key)
 
     def __setitem__(self, key: Any, value: _T) -> None:
         if not isinstance(key, int):
             raise TypeError("Keys must be of type `int`")
-        return super().__setitem__(key, value)  # pylint: disable=no-member
+        return super().__setitem__(key, value)
 
 
 class _RulesDict(Dict[Union[int, str], PermState]):
@@ -939,12 +939,12 @@ class _RulesDict(Dict[Union[int, str], PermState]):
     def __getitem__(self, key: Any) -> PermState:
         if key != Requires.DEFAULT and not isinstance(key, int):
             raise TypeError(f'Expected "{Requires.DEFAULT}" or int key, not "{key}"')
-        return super().__getitem__(key)  # pylint: disable=no-member
+        return super().__getitem__(key)
 
     def __setitem__(self, key: Any, value: PermState) -> None:
         if key != Requires.DEFAULT and not isinstance(key, int):
             raise TypeError(f'Expected "{Requires.DEFAULT}" or int key, not "{key}"')
-        return super().__setitem__(key, value)  # pylint: disable=no-member
+        return super().__setitem__(key, value)
 
 
 def _validate_perms_dict(perms: Dict[str, bool]) -> None:

redbot/core/config.py

@@ -66,7 +66,7 @@ def get_latest_confs() -> Tuple["Config"]:
     return tuple(ret)
 
 
-class _ValueCtxManager(Awaitable[_T], AsyncContextManager[_T]):  # pylint: disable=duplicate-bases
+class _ValueCtxManager(Awaitable[_T], AsyncContextManager[_T]):
     """Context manager implementation of config values.
 
     This class allows mutable config values to be both "get" and "set" from

redbot/core/drivers/_mongo.py

@@ -9,7 +9,6 @@
 from urllib.parse import quote_plus
 
 try:
-    # pylint: disable=import-error
     import motor.core
     import motor.motor_asyncio
     import pymongo.errors

redbot/core/drivers/json.py

@@ -252,7 +252,7 @@ def _save_json(path: Path, data: Dict[str, Any]) -> None:
     tmp_path.replace(path)
 
     try:
-        flag = os.O_DIRECTORY  # pylint: disable=no-member
+        flag = os.O_DIRECTORY
     except AttributeError:
         pass
     else:

redbot/core/drivers/postgres/postgres.py

@@ -5,7 +5,6 @@
 from typing import Any, AsyncIterator, Callable, List, Optional, Tuple, Union
 
 try:
-    # pylint: disable=import-error
     import asyncpg
 except ModuleNotFoundError:
     asyncpg = None