Adds ruff, a drop-in replacement for black

.gitignore

@@ -1,3 +1,4 @@
+# isort: skip_file
 .venv
 __pycache__
 /build

pyproject.toml

@@ -41,7 +41,7 @@ types-protobuf = ">=3.20"
 typing-extensions = "^4.2.0"
 
 [tool.poetry.dev-dependencies]
-black = "^23.1.0"
+ruff = "^0.1.5"
 cibuildwheel = "^2.11.0"
 grpcio-tools = "^1.48.0"
 isort = "^5.11.5"
@@ -68,12 +68,11 @@ grpc = ["grpcio"]
 build-develop = "python scripts/setup_bridge.py develop"
 build-develop-with-release = { cmd = "python scripts/setup_bridge.py develop", env = { TEMPORAL_BUILD_RELEASE = "1" }}
 fix-wheel = "python scripts/fix_wheel.py"
-format = [{cmd = "black ."}, {cmd = "isort ."}]
+format = [{cmd = "ruff format ."}]
 gen-docs = "python scripts/gen_docs.py"
 gen-protos = "python scripts/gen_protos.py"
 lint = [
-  {cmd = "black --check ."},
-  {cmd = "isort --check-only ."},
+  {cmd = "ruff check ."},
   {ref = "lint-types"},
   {ref = "lint-docs"},
 ]
@@ -117,10 +116,6 @@ build-verbosity = "1"
 before-all = "curl https://sh.rustup.rs -sSf | sh -s -- --default-toolchain stable -y && yum install -y openssl-devel"
 environment = { PATH = "$PATH:$HOME/.cargo/bin", CARGO_NET_GIT_FETCH_WITH_CLI = "true" }
 
-[tool.isort]
-profile = "black"
-skip_gitignore = true
-
 [tool.mypy]
 ignore_missing_imports = true
 exclude = [

scripts/gen_protos.py

@@ -6,7 +6,7 @@
 import tempfile
 from functools import partial
 from pathlib import Path
-from typing import List, Mapping, Optional
+from typing import List, Mapping
 
 base_dir = Path(__file__).parent.parent
 proto_dir = base_dir / "temporalio" / "bridge" / "sdk-core" / "protos"
@@ -22,8 +22,8 @@
     v
     for v in proto_dir.glob("**/*.proto")
     if not str(v).startswith(str(testsrv_proto_dir / "dependencies"))
-    and not "health" in str(v)
-    and not "google" in str(v)
+    and "health" not in str(v)
+    and "google" not in str(v)
 ]
 proto_paths.extend(test_proto_dir.glob("**/*.proto"))
 proto_paths.extend(additional_proto_dir.glob("**/*.proto"))
@@ -92,7 +92,7 @@ def fix_generated_output(base_path: Path):
         message_names = sorted(message_names)
         if message_names:
             f.write(
-                f'\n__all__ = [\n    "' + '",\n    "'.join(message_names) + '",\n]\n'
+                '\n__all__ = [\n    "' + '",\n    "'.join(message_names) + '",\n]\n'
             )
         # gRPC imports
         if "service_pb2_grpc" in imports:
@@ -112,7 +112,7 @@ def fix_generated_output(base_path: Path):
                 message_names.append(message)
             # __all__
             message_names = sorted(message_names)
-            f.write(f'    __all__.extend(["' + '", "'.join(message_names) + '"])\n')
+            f.write('    __all__.extend(["' + '", "'.join(message_names) + '"])\n')
             f.write("except ImportError:\n    pass")
 
 

temporalio/activity.py

@@ -247,7 +247,7 @@ def in_activity() -> bool:
     Returns:
         True if in an activity, False otherwise.
     """
-    return not _current_context.get(None) is None
+    return _current_context.get(None) is not None
 
 
 def info() -> Info:
@@ -524,7 +524,8 @@ def _apply_to_callable(
                 fn=fn,
                 # iscoroutinefunction does not return true for async __call__
                 # TODO(cretz): Why can't MyPy handle this?
-                is_async=inspect.iscoroutinefunction(fn) or inspect.iscoroutinefunction(fn.__call__),  # type: ignore
+                is_async=inspect.iscoroutinefunction(fn)
+                or inspect.iscoroutinefunction(fn.__call__),  # type: ignore
                 no_thread_cancel_exception=no_thread_cancel_exception,
             ),
         )

temporalio/api/cluster/v1/message_pb2_grpc.py

@@ -1,3 +1,2 @@
 # Generated by the gRPC Python protocol compiler plugin. DO NOT EDIT!
 """Client and server classes corresponding to protobuf-defined services."""
-import grpc