dev: use ruff instead of black and pylint

Author: lervag

.github/workflows/ci.yaml

@@ -24,8 +24,8 @@ jobs:
       - name: Install the project
         run: uv sync --all-extras --dev
 
-      - name: Run black
-        run: uv run black --check .
+      - name: Run ruff
+        run: uv run ruff format --check
 
       - name: Run mypy
         run: uv run mypy src

README.md

@@ -335,7 +335,7 @@ uv run pytest
 To format the code, run:
 
 ```sh
-uv run black .
+uv run ruff format
 ```
 
 To type-check the code, run:

pyproject.toml

@@ -32,12 +32,11 @@ apy = "apyanki.cli:main"
 [dependency-groups]
 dev = [
     "pytest>=7.3.1",
-    "pylint>=3.0.2",
     "bump2version>=1.0.1",
-    "black>=24.3.0",
     "mypy>=1.3.0",
     "types-beautifulsoup4>=4.12.0.5",
     "types-markdown>=3.4.2.9",
+    "ruff>=0.12.0",
 ]
 
 [build-system]
@@ -47,23 +46,6 @@ build-backend = "hatchling.build"
 [tool.pytest]
 filterwarnings = ["ignore::DeprecationWarning:html5lib.*:"]
 
-[tool.pylint.master]
-init-hook = 'import sys; sys.path.append("/usr/share/anki")'
-
-[tool.pylint.reports]
-output-format = "colorized"
-
-[tool.pylint.format]
-max-line-length = "88"
-
-[tool.pylint."messages control"]
-max-statements = 75
-disable = [
-    "invalid-name",
-    "too-many-branches",
-    "too-many-instance-attributes",
-]
-
 [tool.mypy]
 exclude = ".venv"
 strict = true

src/apyanki/anki.py

@@ -109,7 +109,6 @@ def _init_load_profile(
 
     def _init_load_collection(self) -> None:
         """Load the Anki collection"""
-        # pylint: disable=import-outside-toplevel
         from anki.collection import Collection
         from anki.errors import DBError
 
@@ -132,7 +131,6 @@ def _init_load_collection(self) -> None:
     @staticmethod
     def _init_load_config() -> None:
         """Load custom configuration"""
-        # pylint: disable=import-outside-toplevel
         from anki import latex
 
         # Update LaTeX commands
@@ -160,7 +158,6 @@ def __exit__(
 
     def sync(self) -> None:
         """Sync collection to AnkiWeb"""
-        # pylint: disable=import-outside-toplevel
         from anki.sync import SyncAuth
 
         if self._profile is None:
@@ -230,7 +227,6 @@ def sync(self) -> None:
 
     def check_media(self) -> None:
         """Check media (will rebuild missing LaTeX files)"""
-        # pylint: disable=import-outside-toplevel
         from anki.notes import NoteId
 
         with cd(self.col.media.dir()):
@@ -289,7 +285,6 @@ def delete_notes(self, ids: NoteId | list[NoteId]) -> None:
 
     def get_model(self, model_name: str) -> Optional[NotetypeDict]:
         """Get model from model name"""
-        # pylint: disable=import-outside-toplevel
         from anki.models import NotetypeId
 
         model_id = self.model_name_to_id.get(model_name)

src/apyanki/cli.py

@@ -735,5 +735,4 @@ def backup(target_file: Path, include_media: bool, legacy: bool) -> None:
 
 
 if __name__ == "__main__":
-    # pylint: disable=no-value-for-parameter
     main()

src/apyanki/fields.py

@@ -166,10 +166,8 @@ def img_paths_from_field_latex(html: str, ntd: NotetypeDict, anki: Anki) -> list
 
     Note: The returned paths are relative to the Anki media directory.
     """
-    # pylint: disable=import-outside-toplevel
     from anki import latex
 
-    # pylint: disable=protected-access
     proto = anki.col._backend.extract_latex(
         text=html, svg=ntd.get("latexsvg", False), expand_clozes=False
     )

src/apyanki/note.py

@@ -82,7 +82,6 @@ def __repr__(self) -> str:
 
     def pprint(self, print_raw: bool = False, list_cards: bool = False) -> None:
         """Print to screen"""
-        # pylint: disable=import-outside-toplevel
         from anki import latex
 
         header = f"[green]# Note (nid: {self.n.id})[/green]"

tests/common.py

@@ -1,6 +1,5 @@
 """Implement some basic test fixtures"""
 
-# pylint: disable=too-few-public-methods
 import os
 import tempfile
 import shutil