move and update ruff config

Author: pvsr

pyproject.toml

@@ -44,3 +44,34 @@ ignore_missing_imports = true
 [[tool.mypy.overrides]]
 module = "tests.*"
 disallow_untyped_defs = false
+
+[tool.ruff]
+select = [
+    "E",
+    "F",
+    "W",
+    "I",
+    "UP",
+    "N",
+    "ANN",
+    "B",
+    "A",
+    "C4",
+    "PT",
+    "SIM",
+    "ARG",
+    "PTH",
+    "PL",
+    "RUF",
+]
+target-version = "py39"
+ignore = [
+  "E501", # line too long
+  "ANN101", # annotate self
+  "ANN102", # annotate cls
+  "ANN401", # ban Any
+]
+
+[tool.ruff.per-file-ignores]
+"tests/test_main.py" = [ "S101", "ANN201"]
+"tests/test_profiles.py" = [ "S101", "ANN201"]

qbpm/main.py

@@ -1,3 +1,4 @@
+import sys
 from pathlib import Path
 from typing import Any, Callable, NoReturn, Optional
 
@@ -139,7 +140,7 @@ def edit(profile_dir: Path, profile_name: str) -> None:
     profile = Profile(profile_name, profile_dir)
     if not profile.exists():
         error(f"profile {profile.name} not found at {profile.root}")
-        exit(1)
+        sys.exit(1)
     click.edit(filename=str(profile.root / "config" / "config.py"))
 
 
@@ -178,10 +179,10 @@ def session_info(
     if not session_path:
         tried = ", ".join([str(p.resolve()) for p in session_paths])
         error(f"could not find session at the following paths: {tried}")
-        exit(1)
+        sys.exit(1)
 
     return (Profile(profile_name or session_path.stem, profile_dir), session_path)
 
 
 def exit_with(result: bool) -> NoReturn:
-    exit(0 if result else 1)
+    sys.exit(0 if result else 1)

qbpm/utils.py

@@ -10,7 +10,7 @@
 WAYLAND_MENUS = ["fuzzel", "wofi", "dmenu-wl"]
 X11_MENUS = ["rofi", "dmenu"]
 AUTO_MENUS = WAYLAND_MENUS + X11_MENUS
-SUPPORTED_MENUS = AUTO_MENUS + ["fzf", "applescript"]
+SUPPORTED_MENUS = [*AUTO_MENUS, "fzf", "applescript"]
 
 
 def error(msg: str) -> None:

ruff.toml

@@ -67,7 +67,7 @@ def test_overwrite_config(tmp_path: Path):
     profiles.create_config(profile)
     profiles.create_config(profile, url, True)
     assert list(config_dir.iterdir()) == [config_dir / "config.py"]
-    with open(config_dir / "config.py") as conf:
+    with (config_dir / "config.py").open() as conf:
         for line in conf:
             if url in line:
                 return