Run black with experimental string processing (#1725)

Author: BobDotCom

.pre-commit-config.yaml

@@ -27,11 +27,12 @@ repos:
     rev: 22.10.0
     hooks:
       - id: black
+        # See https://github.com/psf/black/issues/2188#issuecomment-1289317647 for why we can't use the --preview flag.
         args: [--safe, --quiet]
-  #  - repo: https://github.com/Pierre-Sassoulas/black-disable-checker
-  #    rev: 1.0.1
-  #    hooks:
-  #      - id: black-disable-checker
+  - repo: https://github.com/Pierre-Sassoulas/black-disable-checker
+    rev: 1.0.1
+    hooks:
+      - id: black-disable-checker
   #  - repo: https://github.com/PyCQA/flake8
   #    rev: 4.0.1
   #    hooks:

discord/abc.py

@@ -449,7 +449,8 @@ async def _edit(
             for target, perm in overwrites.items():
                 if not isinstance(perm, PermissionOverwrite):
                     raise InvalidArgument(
-                        f"Expected PermissionOverwrite received {perm.__class__.__name__}"
+                        "Expected PermissionOverwrite received"
+                        f" {perm.__class__.__name__}"
                     )
 
                 allow, deny = perm.pair()
@@ -1516,7 +1517,8 @@ async def send(
                 reference = reference.to_message_reference_dict()
             except AttributeError:
                 raise InvalidArgument(
-                    "reference parameter must be Message, MessageReference, or PartialMessage"
+                    "reference parameter must be Message, MessageReference, or"
+                    " PartialMessage"
                 ) from None
 
         if view:

discord/activity.py

@@ -664,7 +664,10 @@ def __str__(self) -> str:
         return "Spotify"
 
     def __repr__(self) -> str:
-        return f"<Spotify title={self.title!r} artist={self.artist!r} track_id={self.track_id!r}>"
+        return (
+            "<Spotify"
+            f" title={self.title!r} artist={self.artist!r} track_id={self.track_id!r}>"
+        )
 
     @property
     def title(self) -> str:
@@ -790,7 +793,8 @@ def __init__(
             self.emoji = emoji
         else:
             raise TypeError(
-                f"Expected str, PartialEmoji, or None, received {type(emoji)!r} instead."
+                "Expected str, PartialEmoji, or None, received"
+                f" {type(emoji)!r} instead."
             )
 
     @property

discord/bot.py

@@ -505,7 +505,8 @@ def register(
             if kwargs.pop("_log", True):
                 if method == "bulk":
                     _log.debug(
-                        f"Bulk updating commands {[c['name'] for c in args[0]]} for guild {guild_id}"
+                        f"Bulk updating commands {[c['name'] for c in args[0]]} for"
+                        f" guild {guild_id}"
                     )
                 # TODO: Find where "cmd" is defined
                 elif method == "upsert":
@@ -636,7 +637,8 @@ def register(
             )
             if not cmd:
                 raise ValueError(
-                    f"Registered command {i['name']}, type {i.get('type')} not found in pending commands"
+                    f"Registered command {i['name']}, type {i.get('type')} not found in"
+                    " pending commands"
                 )
             cmd.id = i["id"]
             self._application_commands[cmd.id] = cmd
@@ -831,7 +833,6 @@ async def process_application_commands(
                     if guild_id is None:
                         await self.sync_commands()
                     else:
-
                         await self.sync_commands(check_guilds=[guild_id])
                 return self._bot.dispatch("unknown_application_command", interaction)
 

discord/channel.py

@@ -2061,7 +2061,10 @@ def __init__(
         self._update(guild, data)
 
     def __repr__(self) -> str:
-        return f"<CategoryChannel id={self.id} name={self.name!r} position={self.position} nsfw={self.nsfw}>"
+        return (
+            "<CategoryChannel"
+            f" id={self.id} name={self.name!r} position={self.position} nsfw={self.nsfw}>"
+        )
 
     def _update(self, guild: Guild, data: CategoryChannelPayload) -> None:
         # This data will always exist

discord/client.py

@@ -568,7 +568,6 @@ async def connect(self, *, reconnect: bool = True) -> None:
                 aiohttp.ClientError,
                 asyncio.TimeoutError,
             ) as exc:
-
                 self.dispatch("disconnect")
                 if not reconnect:
                     await self.close()
@@ -1768,7 +1767,8 @@ def add_view(self, view: View, *, message_id: int | None = None) -> None:
 
         if not view.is_persistent():
             raise ValueError(
-                "View is not persistent. Items need to have a custom_id set and View must have no timeout"
+                "View is not persistent. Items need to have a custom_id set and View"
+                " must have no timeout"
             )
 
         self._connection.store_view(view, message_id)

discord/cog.py

@@ -143,7 +143,10 @@ def __new__(cls: type[CogMeta], *args: Any, **kwargs: Any) -> CogMeta:
 
         commands = {}
         listeners = {}
-        no_bot_cog = "Commands or listeners must not start with cog_ or bot_ (in method {0.__name__}.{1})"
+        no_bot_cog = (
+            "Commands or listeners must not start with cog_ or bot_ (in method"
+            " {0.__name__}.{1})"
+        )
 
         new_cls = super().__new__(cls, name, bases, attrs, **kwargs)
 
@@ -177,7 +180,8 @@ def __new__(cls: type[CogMeta], *args: Any, **kwargs: Any) -> CogMeta:
                 if isinstance(value, _filter):
                     if is_static_method:
                         raise TypeError(
-                            f"Command in method {base}.{elem!r} must not be staticmethod."
+                            f"Command in method {base}.{elem!r} must not be"
+                            " staticmethod."
                         )
                     if elem.startswith(("cog_", "bot_")):
                         raise TypeError(no_bot_cog.format(base, elem))
@@ -187,7 +191,8 @@ def __new__(cls: type[CogMeta], *args: Any, **kwargs: Any) -> CogMeta:
                 if hasattr(value, "add_to") and not getattr(value, "parent", None):
                     if is_static_method:
                         raise TypeError(
-                            f"Command in method {base}.{elem!r} must not be staticmethod."
+                            f"Command in method {base}.{elem!r} must not be"
+                            " staticmethod."
                         )
                     if elem.startswith(("cog_", "bot_")):
                         raise TypeError(no_bot_cog.format(base, elem))
@@ -386,7 +391,8 @@ def listener(cls, name: str = MISSING) -> Callable[[FuncT], FuncT]:
 
         if name is not MISSING and not isinstance(name, str):
             raise TypeError(
-                f"Cog.listener expected str but received {name.__class__.__name__!r} instead."
+                "Cog.listener expected str but received"
+                f" {name.__class__.__name__!r} instead."
             )
 
         def decorator(func: FuncT) -> FuncT:

discord/commands/context.py

@@ -294,7 +294,8 @@ def send_response(self) -> Callable[..., Awaitable[Interaction]]:
             return self.interaction.response.send_message
         else:
             raise RuntimeError(
-                f"Interaction was already issued a response. Try using {type(self).__name__}.send_followup() instead."
+                "Interaction was already issued a response. Try using"
+                f" {type(self).__name__}.send_followup() instead."
             )
 
     @property
@@ -304,7 +305,8 @@ def send_followup(self) -> Callable[..., Awaitable[WebhookMessage]]:
             return self.followup.send
         else:
             raise RuntimeError(
-                f"Interaction was not yet issued a response. Try using {type(self).__name__}.respond() first."
+                "Interaction was not yet issued a response. Try using"
+                f" {type(self).__name__}.respond() first."
             )
 
     @property

discord/commands/core.py

@@ -374,7 +374,6 @@ async def invoke(self, ctx: ApplicationContext) -> None:
         await injected(ctx)
 
     async def can_run(self, ctx: ApplicationContext) -> bool:
-
         if not await ctx.bot.can_run(ctx):
             raise CheckFailure(
                 f"The global check functions for command {self.name} failed."
@@ -1822,7 +1821,8 @@ def validate_chat_input_name(name: Any, locale: str | None = None):
     # Must meet the regex ^[-_\w\d\u0901-\u097D\u0E00-\u0E7F]{1,32}$
     if locale is not None and locale not in valid_locales:
         raise ValidationError(
-            f"Locale '{locale}' is not a valid locale, see {docs}/reference#locales for list of supported locales."
+            f"Locale '{locale}' is not a valid locale, see {docs}/reference#locales for"
+            " list of supported locales."
         )
     error = None
     if not isinstance(name, str):
@@ -1831,8 +1831,10 @@ def validate_chat_input_name(name: Any, locale: str | None = None):
         )
     elif not re.match(r"^[-_\w\d\u0901-\u097D\u0E00-\u0E7F]{1,32}$", name):
         error = ValidationError(
-            r"Command names and options must follow the regex \"^[-_\w\d\u0901-\u097D\u0E00-\u0E7F]{1,32}$\". "
-            f"For more information, see {docs}/interactions/application-commands#application-command-object-"
+            r"Command names and options must follow the regex"
+            r" \"^[-_\w\d\u0901-\u097D\u0E00-\u0E7F]{1,32}$\". "
+            "For more information, see"
+            f" {docs}/interactions/application-commands#application-command-object-"
             f'application-command-naming. Received "{name}"'
         )
     elif (
@@ -1851,16 +1853,19 @@ def validate_chat_input_name(name: Any, locale: str | None = None):
 def validate_chat_input_description(description: Any, locale: str | None = None):
     if locale is not None and locale not in valid_locales:
         raise ValidationError(
-            f"Locale '{locale}' is not a valid locale, see {docs}/reference#locales for list of supported locales."
+            f"Locale '{locale}' is not a valid locale, see {docs}/reference#locales for"
+            " list of supported locales."
         )
     error = None
     if not isinstance(description, str):
         error = TypeError(
-            f'Command and option description must be of type str. Received "{description}"'
+            "Command and option description must be of type str. Received"
+            f' "{description}"'
         )
     elif not 1 <= len(description) <= 100:
         error = ValidationError(
-            f'Command and option description must be 1-100 characters long. Received "{description}"'
+            "Command and option description must be 1-100 characters long. Received"
+            f' "{description}"'
         )
 
     if error:

discord/commands/options.py

@@ -278,11 +278,13 @@ def __init__(
 
         if self.min_value is not None and not isinstance(self.min_value, minmax_types):
             raise TypeError(
-                f'Expected {minmax_typehint} for min_value, got "{type(self.min_value).__name__}"'
+                f"Expected {minmax_typehint} for min_value, got"
+                f' "{type(self.min_value).__name__}"'
             )
         if self.max_value is not None and not isinstance(self.max_value, minmax_types):
             raise TypeError(
-                f'Expected {minmax_typehint} for max_value, got "{type(self.max_value).__name__}"'
+                f"Expected {minmax_typehint} for max_value, got"
+                f' "{type(self.max_value).__name__}"'
             )
 
         if self.min_length is not None: