🔀 Merge pull request #197 from davep/tighten-linting

Tighten up the linting rules

Author: davep

Makefile

@@ -8,7 +8,7 @@ publish := uv publish --username=__token__ --keyring-provider=subprocess
 test    := $(run) pytest
 python  := $(run) python
 ruff    := $(run) ruff
-lint    := $(ruff) check --select I
+lint    := $(ruff) check
 fmt     := $(ruff) format
 mypy    := $(run) mypy
 spell   := $(run) codespell

pyproject.toml

@@ -89,3 +89,26 @@ asyncio_default_fixture_loop_scope = "function"
 venvPath="."
 venv=".venv"
 exclude=[".venv"]
+
+[tool.ruff.lint]
+select = [
+    # pycodestyle
+    "E",
+    # Pyflakes
+    "F",
+    # pyupgrade
+    "UP",
+    # flake8-bugbear
+    "B",
+    # flake8-simplify
+    "SIM",
+    # isort
+    "I",
+]
+ignore = [
+    # I think try...expect...pass reads far better.
+    "SIM105",
+]
+
+[tool.ruff.lint.pycodestyle]
+max-line-length = 120

src/braindrop/app/braindrop.py

@@ -97,7 +97,7 @@ def api_token(self) -> str | None:
             return self.environmental_token() or token_file().read_text(
                 encoding="utf-8"
             )
-        except IOError:
+        except OSError:
             pass
         return None
 

src/braindrop/app/data/config.py

@@ -2,12 +2,12 @@
 
 ##############################################################################
 # Python imports.
+from collections.abc import Iterator
 from contextlib import contextmanager
 from dataclasses import asdict, dataclass, field
-from functools import lru_cache
+from functools import cache
 from json import dumps, loads
 from pathlib import Path
-from typing import Iterator
 
 ##############################################################################
 # Local imports.
@@ -63,7 +63,7 @@ def save_configuration(configuration: Configuration) -> Configuration:
 
 
 ##############################################################################
-@lru_cache(maxsize=None)
+@cache
 def load_configuration() -> Configuration:
     """Load the configuration.
 

src/braindrop/app/data/local.py

@@ -2,10 +2,11 @@
 
 ##############################################################################
 # Python imports.
+from collections.abc import Callable, Iterable, Iterator
 from datetime import datetime
 from json import dumps, loads
 from pathlib import Path
-from typing import Any, Callable, Final, Iterable, Iterator, Self
+from typing import Any, Final, Self
 
 ##############################################################################
 # pytz imports.

src/braindrop/app/data/raindrops.py

@@ -6,9 +6,11 @@
 
 ##############################################################################
 # Python imports.
+from collections import Counter
+from collections.abc import Callable, Iterable, Iterator
 from dataclasses import dataclass
 from functools import total_ordering
-from typing import Callable, Counter, Iterable, Iterator, Self, TypeAlias
+from typing import Self
 
 ##############################################################################
 # Local imports.
@@ -81,7 +83,7 @@ def __eq__(self, value: object, /) -> bool:
 
 
 ##############################################################################
-Filters: TypeAlias = tuple["Filter", ...]
+type Filters = tuple["Filter", ...]
 """The type of a collection of filters."""
 
 

src/braindrop/app/screens/main.py

@@ -2,7 +2,7 @@
 
 ##############################################################################
 # Python imports.
-from typing import Callable
+from collections.abc import Callable
 from webbrowser import open as open_url
 
 ##############################################################################

src/braindrop/app/screens/raindrop_input.py

@@ -2,7 +2,7 @@
 
 ##############################################################################
 # Python imports.
-from typing import Iterator
+from collections.abc import Iterator
 
 ##############################################################################
 # httpx imports.

src/braindrop/app/suggestions/tags.py

@@ -2,8 +2,9 @@
 
 ##############################################################################
 # Python imports.
-import re
-from typing import Final, Iterable, Pattern
+from collections.abc import Iterable
+from re import Pattern, compile
+from typing import Final
 
 ##############################################################################
 # Textual imports.
@@ -33,7 +34,7 @@ def __init__(self, tags: Iterable[Tag | TagCount], use_cache: bool = True) -> No
         self._tags = [tag.tag if isinstance(tag, TagCount) else tag for tag in tags]
         """The tags to take suggestions from."""
 
-    _SUGGESTABLE: Final[Pattern[str]] = re.compile(r".*[^,\s]$")
+    _SUGGESTABLE: Final[Pattern[str]] = compile(r".*[^,\s]$")
     """Regular expression to test if a value deserves a suggestion."""
 
     async def get_suggestion(self, value: str) -> str | None:

src/braindrop/app/widgets/raindrop_details.py

@@ -2,8 +2,9 @@
 
 ##############################################################################
 # Python imports.
+from collections.abc import Callable
 from datetime import datetime
-from typing import Any, Callable, Final
+from typing import Any, Final
 
 ##############################################################################
 # Humanize imports.