lint: Replace Black with Ruff.

Signed-off-by: Anders Kaseorg <[email protected]>

Author: andersk

pyproject.toml

@@ -110,4 +110,5 @@ src = [
     "zulip_bots",
     "zulip_botserver",
 ]
+target-version = "py38"
 line-length = 100

requirements.txt

@@ -1,6 +1,5 @@
 crayons
 twine
-black~=23.10.1
 mock
 pytest
 pytest-cov

tools/deploy

@@ -59,9 +59,7 @@ def pack(options: argparse.Namespace) -> None:
         [deploy]
         bot={}
         zuliprc=zuliprc
-        """.format(
-            options.main
-        )
+        """.format(options.main)
     )
     zip_file.writestr("config.ini", bot_config)
     zip_file.close()

tools/lint

@@ -1,7 +1,6 @@
 #! /usr/bin/env python3
 
 import argparse
-import re
 import sys
 
 from zulint.command import LinterConfig, add_default_linter_arguments
@@ -36,16 +35,14 @@ def run() -> None:
         "ruff", ["ruff", "check", "--quiet"], ["py"], fix_arg="--fix", description="Python linter"
     )
     linter_config.external_linter(
-        "gitlint", ["tools/lint-commits"], description="Git Lint for commit messages"
+        "ruff-format",
+        ["ruff", "format", "--quiet"],
+        ["py"],
+        check_arg="--check",
+        description="Python formatter",
     )
     linter_config.external_linter(
-        "black",
-        ["black"],
-        ["py"],
-        description="Reformats Python code",
-        check_arg=["--check"],
-        suppress_line=lambda line: line == "All done! ✨ 🍰 ✨\n"
-        or re.fullmatch(r"\d+ files? would be left unchanged\.\n", line) is not None,
+        "gitlint", ["tools/lint-commits"], description="Git Lint for commit messages"
     )
 
     @linter_config.lint

tools/provision

@@ -45,8 +45,10 @@ the Python version this command is executed with."""
 
     if py_version <= (3, 1) and (not options.force):
         print(
-            red + "Provision failed: Cannot create venv with outdated Python version ({}).\n"
-            "Maybe try `python3 tools/provision`.".format(py_version_output.strip()) + end_format
+            red
+            + "Provision failed: Cannot create venv with outdated Python version ({}).\n"
+            "Maybe try `python3 tools/provision`.".format(py_version_output.strip())
+            + end_format
         )
         sys.exit(1)
 

zulip/integrations/jabber/jabber_mirror_backend.py

@@ -301,9 +301,7 @@ def config_error(msg: str) -> None:
 zulip configuration file under the jabber_mirror section (exceptions are noted
 in their help sections).  Keys have the same name as options with hyphens
 replaced with underscores.  Zulip configuration options go in the api section,
-as normal.""".replace(
-            "\n", " "
-        )
+as normal.""".replace("\n", " ")
     )
     parser.add_option(
         "--mode",
@@ -314,9 +312,7 @@ def config_error(msg: str) -> None:
 all messages they send on Zulip to Jabber and all private Jabber messages to
 Zulip.  In "public" mode, the mirror uses the credentials for a dedicated mirror
 user and mirrors messages sent to Jabber rooms to Zulip.  Defaults to
-"personal"'''.replace(
-            "\n", " "
-        ),
+"personal"'''.replace("\n", " "),
     )
     parser.add_option(
         "--zulip-email-suffix",
@@ -327,9 +323,7 @@ def config_error(msg: str) -> None:
 suffix before sending requests to the Jabber server.  For example, specifying
 "+foo" will cause messages that are sent to the "bar" room by nickname "qux" to
 be mirrored to the "bar/xmpp" stream in Zulip by user "[email protected]". This
-option does not affect login credentials.""".replace(
-            "\n", " "
-        ),
+option does not affect login credentials.""".replace("\n", " "),
     )
     parser.add_option(
         "-d",

zulip/integrations/perforce/zulip_change-commit.py

@@ -84,9 +84,7 @@
 ```quote
 {desc}
 ```
-""".format(
-    user=metadata["user"], change=change, path=changeroot, desc=metadata["desc"]
-)
+""".format(user=metadata["user"], change=change, path=changeroot, desc=metadata["desc"])
 
 message_data: Dict[str, Any] = {
     "type": "stream",

zulip/tests/test_default_arguments.py

@@ -41,8 +41,9 @@ def test_config_path_with_tilde(self, mock_os_path_exists: bool) -> None:
         expanded_test_path = os.path.abspath(os.path.expanduser(test_path))
         self.assertEqual(
             str(cm.exception),
-            "api_key or email not specified and "
-            "file {} does not exist".format(expanded_test_path),
+            "api_key or email not specified and " "file {} does not exist".format(
+                expanded_test_path
+            ),
         )
 
 

zulip_bots/zulip_bots/bots/beeminder/test_beeminder.py

@@ -79,9 +79,7 @@ def test_invalid_when_handle_message(self) -> None:
             {"auth_token": "someInvalidKey", "username": "aaron", "goalname": "goal"}
         ), patch("requests.get", side_effect=ConnectionError()), self.mock_http_conversation(
             "test_invalid_when_handle_message"
-        ), patch(
-            "logging.exception"
-        ):
+        ), patch("logging.exception"):
             self.verify_reply("5", "Error. Check your key!")
 
     def test_error(self) -> None:

zulip_bots/zulip_bots/bots/chessbot/chessbot.py

@@ -54,8 +54,7 @@ def handle_message(self, message: Dict[str, str], bot_handler: BotHandler) -> No
         if bot_handler.storage.contains("is_with_computer"):
             is_with_computer = (
                 # `bot_handler`'s `storage` only accepts `str` values.
-                bot_handler.storage.get("is_with_computer")
-                == str(True)
+                bot_handler.storage.get("is_with_computer") == str(True)
             )
 
         if bot_handler.storage.contains("last_fen"):