[pre-commit.ci] pre-commit autoupdate (#240)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Josh Karpel <josh.karpel@gmail.com>

.github/workflows/quality-check.yml

@@ -32,7 +32,7 @@ jobs:
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install Poetry
-        uses: snok/install-poetry@v1.3.3
+        uses: snok/install-poetry@v1.3.4
       - name: Install Package
         run: poetry install
       - name: Run pre-commit checks

.pre-commit-config.yaml

@@ -27,18 +27,18 @@ repos:
       - id: python-use-type-annotations
       - id: python-check-blanket-type-ignore
   - repo: https://github.com/python-jsonschema/check-jsonschema
-    rev: 0.23.2
+    rev: 0.27.0
     hooks:
       - id: check-github-workflows
       - id: check-github-actions
       - id: check-dependabot
-  - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: 'v0.0.272'
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: 'v0.0.292'
     hooks:
       - id: ruff
         args: [ --fix, --exit-non-zero-on-fix ]
   - repo: https://github.com/psf/black
-    rev: 23.3.0
+    rev: 23.9.1
     hooks:
       - id: black
   - repo: local

docs/transitions.md

@@ -65,13 +65,13 @@ the [`Transition`][spiel.Transition] [protocol](https://docs.python.org/3/librar
 The protocol is:
 
 ```python title="Transition Protocol"
---8<-- "../spiel/transitions/protocol.py"
+--8<-- "spiel/transitions/protocol.py"
 ```
 
 As an example, consider the [`Swipe`][spiel.Swipe] transition included in Spiel:
 
 ```python title="Swipe Transition"
---8<-- "../spiel/transitions/swipe.py"
+--8<-- "spiel/transitions/swipe.py"
 ```
 
 The transition effect is implemented using

mkdocs.yml

@@ -61,7 +61,7 @@ markdown_extensions:
       anchor_linenums: true
   - pymdownx.inlinehilite
   - pymdownx.snippets:
-      base_path: ['docs']
+      base_path: ['docs', '.']
       check_paths: true
   - pymdownx.superfences
   - pymdownx.tabbed:

spiel/app.py

@@ -11,7 +11,7 @@
 from functools import cached_property, partial
 from pathlib import Path
 from time import monotonic
-from typing import Callable, ContextManager, Iterator
+from typing import Callable, ClassVar, ContextManager, Iterator, List, Tuple
 
 from rich.style import Style
 from rich.text import Text
@@ -67,7 +67,7 @@ def load_deck(path: Path) -> Deck:
 
 class SpielApp(App[None]):
     CSS_PATH = "spiel.css"
-    BINDINGS = [
+    BINDINGS: ClassVar[List[Binding | Tuple[str, str, str]]] = [
         Binding("d", "switch_screen('deck')", "Go to the Deck view."),
         Binding("question_mark", "push_screen('help')", "Go to the Help view."),
         Binding("i", "repl", "Switch to the REPL."),

spiel/screens/deck.py

@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+from typing import ClassVar, List, Tuple
+
 from textual.app import ComposeResult
 from textual.binding import Binding
 
@@ -9,7 +11,7 @@
 
 
 class DeckScreen(SpielScreen):
-    BINDINGS = [
+    BINDINGS: ClassVar[List[Binding | Tuple[str, str, str]]] = [
         Binding("right", "next_slide", "Go to next slide."),
         Binding("left", "prev_slide", "Go to previous slide."),
         Binding("down", "next_row", "Go to next row of slides."),

spiel/screens/help.py

@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+from typing import ClassVar, List, Tuple
+
 from textual.app import ComposeResult
 from textual.binding import Binding
 from textual.containers import Container
@@ -19,7 +21,7 @@ class HelpScreen(SpielScreen):
     }
     """
 
-    BINDINGS = [
+    BINDINGS: ClassVar[List[Binding | Tuple[str, str, str]]] = [
         Binding("escape,enter", "pop_screen", "Return to the previous view."),
     ]
 

spiel/screens/slide.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import inspect
+from typing import ClassVar, List, Tuple
 
 from textual.app import ComposeResult
 from textual.binding import Binding
@@ -14,7 +15,7 @@
 
 
 class SlideScreen(SpielScreen):
-    BINDINGS = [
+    BINDINGS: ClassVar[List[Binding | Tuple[str, str, str]]] = [
         Binding("right", "next_slide", "Go to next slide."),
         Binding("left", "prev_slide", "Go to previous slide."),
         Binding("t", "trigger", "Trigger the current slide."),

tests/cli/test_help.py

@@ -14,6 +14,6 @@ def test_help(runner: CliRunner) -> None:
 
 
 def test_help_via_main() -> None:
-    result = subprocess.run([sys.executable, "-m", PACKAGE_NAME, "--help"])
+    result = subprocess.run([sys.executable, "-m", PACKAGE_NAME, "--help"], check=False)
 
     assert result.returncode == 0