Add ruff pre-commit

.pre-commit-config.yaml

@@ -9,3 +9,10 @@ repos:
     rev: 22.12.0
     hooks:
     -   id: black
+- repo: https://github.com/charliermarsh/ruff-pre-commit
+  # Ruff version.
+  rev: 'v0.0.199'
+  hooks:
+    - id: ruff
+      # Respect `exclude` and `extend-exclude` settings.
+      args: ["--force-exclude"]

plugins/cell/txl_cell/components.py

@@ -24,7 +24,7 @@ def source(self, value: List[str]):
     def outputs(self) -> List[Dict[str:Any]]:
         return self.outputs
 
-    @source.setter
+    @outputs.setter
     def outputs(self, value: List[Dict[str:Any]]):
         self.outputs = value
 

plugins/editors/txl_editors/__init__.py

@@ -1,3 +1 @@
 __version__ = "0.1.4"
-
-from .components import Editors

plugins/file_browser/txl_filebrowser/components.py

@@ -77,7 +77,8 @@ def __init__(
         )
 
     def process_label(self, label: TextType):
-        """Process a str or Text in to a label. Maybe overridden in a subclass to change modify how labels are rendered.
+        """Process a str or Text in to a label. Maybe overridden in a subclass to change
+        modify how labels are rendered.
         Args:
             label (TextType): Label.
         Returns:

plugins/jpterm/txl_jpterm/main_area.py

@@ -1,5 +1,4 @@
 from textual.widget import Widget
-from textual.widgets import Static
 
 from txl.base import MainArea as AbstractMainArea
 

plugins/launcher/txl_launcher/components.py

@@ -1,5 +1,4 @@
 from asphalt.core import Component, Context
-from textual.app import ComposeResult
 from textual.widget import Widget
 from textual.widgets import Button
 from txl.base import Launcher, MainArea

plugins/notebook/txl_notebook/components.py

@@ -1,5 +1,3 @@
-from typing import Any, Dict, List
-
 from asphalt.core import Component, Context
 from jupyter_ydoc import YNotebook
 from txl.base import Notebook, NotebookFactory

plugins/notebook_editor/txl_notebook_editor/components.py

@@ -1,5 +1,3 @@
-from pathlib import Path
-
 from asphalt.core import Component, Context
 from textual.binding import Binding
 from textual.containers import Container

plugins/notebook_viewer/txl_notebook_viewer/components.py

@@ -54,12 +54,13 @@ def update_viewer(self):
         )
         theme = "ansi_dark"
 
-        if not "cells" in self.nb:
+        if "cells" not in self.nb:
             return
 
         for cell in self.nb["cells"]:
             execution_count = (
-                f"[green]In [[#66ff00]{cell['execution_count'] or ' '}[/#66ff00]]:[/green]"
+                f"[green]In [[#66ff00]{cell['execution_count'] or ' '}"
+                "[/#66ff00]]:[/green]"
                 if "execution_count" in cell
                 else ""
             )

plugins/remote_contents/txl_remote_contents/components.py

@@ -29,7 +29,7 @@ def __aiter__(self):
     async def __anext__(self) -> bytes:
         try:
             message = await self.recv()
-        except:
+        except BaseException:
             raise StopAsyncIteration()
         return message
 

plugins/text_viewer/txl_text_viewer/components.py

@@ -1,7 +1,6 @@
 from asphalt.core import Component, Context
 from rich.syntax import Syntax
 from rich.traceback import Traceback
-from textual.scroll_view import ScrollView
 from textual.widgets import Static
 from txl.base import Editor, Editors, Contents, FileOpenEvent
 from txl.hooks import register_component

txl/txl/base.py

@@ -68,7 +68,7 @@ def source(self, value: List[str]):
     def outputs(self) -> List[Dict[str, Any]]:
         ...
 
-    @source.setter
+    @outputs.setter
     @abstractmethod
     def outputs(self, value: List[Dict[str, Any]]):
         ...