fix linting errors

Author: LilSpazJoekp

discord_slash/__init__.py

@@ -8,11 +8,11 @@
 :license: MIT
 """
 
-from .client import SlashCommand
-from .model import SlashCommandOptionType
-from .context import SlashContext
-from .context import ComponentContext
-from .dpy_overrides import ComponentMessage
-from .utils import manage_commands
-from .utils import manage_components
-from .const import __version__
+from .client import SlashCommand  # noqa: F401
+from .model import SlashCommandOptionType  # noqa: F401
+from .context import SlashContext  # noqa: F401
+from .context import ComponentContext  # noqa: F401
+from .dpy_overrides import ComponentMessage  # noqa: F401
+from .utils import manage_commands  # noqa: F401
+from .utils import manage_components  # noqa: F401
+from .const import __version__  # noqa: F401

discord_slash/client.py

@@ -9,7 +9,6 @@
 from . import model
 from . import error
 from . import context
-from . import dpy_overrides
 from .utils import manage_commands
 
 
@@ -336,7 +335,7 @@ async def sync_all_commands(
             for cmd in existing_cmds:
                 existing_by_name[cmd["name"]] = model.CommandData(**cmd)
 
-            if len(new_cmds) != len(existing_cmds): 
+            if len(new_cmds) != len(existing_cmds):
                 changed = True
 
             for command in new_cmds:
@@ -356,7 +355,7 @@ async def sync_all_commands(
                     changed=True
                     to_send.append(command)
 
-            
+
             if changed:
                 self.logger.debug(f"Detected changes on {scope if scope is not None else 'global'}, updating them")
                 existing_cmds = await self.req.put_slash_commands(slash_commands=to_send, guild_id=scope)
@@ -401,7 +400,7 @@ async def sync_all_commands(
                     if existing_perms_model[new_perm["id"]] != model.GuildPermissionsData(**new_perm):
                         changed = True
                         break
-            
+
             if changed:
                 self.logger.debug(f"Detected permissions changes on {scope}, updating them")
                 await self.req.update_guild_commands_permissions(scope, new_perms)
@@ -414,7 +413,7 @@ async def sync_all_commands(
             other_guilds = [guild.id for guild in self._discord.guilds if guild.id not in cmds["guild"]]
             # This is an extremly bad way to do this, because slash cmds can be in guilds the bot isn't in
             # But it's the only way until discord makes an endpoint to request all the guild with cmds registered.
-            
+
             for guild in other_guilds:
                 with suppress(discord.Forbidden):
                     existing = await self.req.get_all_commands(guild_id = guild)
@@ -772,11 +771,11 @@ def wrapper(cmd):
     def permission(self, guild_id: int, permissions: list):
         """
         Decorator that add permissions. This will set the permissions for a single guild, you can use it more than once for each command.
-        :param guild_id: ID of the guild for the permissions. 
+        :param guild_id: ID of the guild for the permissions.
         :type guild_id: int
         :param permissions: Permission requirements of the slash command. Default ``None``.
         :type permissions: dict
-        
+
         """
         def wrapper(cmd):
             if not getattr(cmd, "__permissions__", None):

discord_slash/context.py

@@ -338,12 +338,11 @@ async def edit_origin(self, **fields):
             if self.deferred:
                 _json = await self._http.edit(_resp, self._token, files=files)
                 self.deferred = False
-            else:
-                json_data = {
-                    "type": 7,
-                    "data": _resp
-                }
-                _json = await self._http.post_initial_response(json_data, self.interaction_id, self._token)
+            else:  # noqa: F841
+                json_data = {"type": 7, "data": _resp}
+                _json = await self._http.post_initial_response(  # noqa: F841
+                    json_data, self.interaction_id, self._token
+                )
             self.responded = True
         else:
             raise error.IncorrectFormat("Already responded")

discord_slash/http.py

@@ -131,7 +131,7 @@ def post_initial_response(self, _resp, interaction_id, token):
         return self.command_response(token, False, "POST", interaction_id, json=_resp)
 
     def command_response(self, token, use_webhook, method, interaction_id= None, url_ending = "", **kwargs):
-        """
+        r"""
         Sends a command response to discord (POST, PATCH, DELETE)
 
         :param token: Interaction token

discord_slash/model.py

@@ -177,7 +177,7 @@ async def _concurrency_checks(self, ctx):
         try:
             # cooldown checks
             self._prepare_cooldowns(ctx)
-        except:
+        except Exception:
             if self._max_concurrency is not None:
                 await self._max_concurrency.release(ctx)
             raise
@@ -498,7 +498,7 @@ class GuildPermissionsData:
     Slash permissions data for a command in a guild.
 
     :ivar id: Command id, provided by discord.
-    :ivar guild_id: Guild id that the permissions are in. 
+    :ivar guild_id: Guild id that the permissions are in.
     :ivar permissions: List of permissions dict.
     """
     def __init__(self, id, guild_id, permissions, **kwargs):

discord_slash/utils/manage_commands.py

@@ -269,7 +269,7 @@ def create_option(name: str,
 
     .. note::
         ``choices`` must either be a list of `option type dicts <https://discord.com/developers/docs/interactions/slash-commands#applicationcommandoptionchoice>`_
-        or a list of single string values. 
+        or a list of single string values.
     """
     if not isinstance(option_type, int) or isinstance(option_type, bool): #Bool values are a subclass of int
         original_type = option_type
@@ -320,7 +320,7 @@ def generate_options(function: Callable, description: str = "No description.", c
             args = getattr(param.annotation, "__args__", None)
             if args:
                 param = param.replace(annotation=args[0])
-                required = not args[-1] is type(None)
+                required = not isinstance(args[-1], type(None))
 
         option_type = SlashCommandOptionType.from_type(param.annotation) or SlashCommandOptionType.STRING
         name = param.name if not connector else connector[param.name]