Merge pull request #23 from wli51/dev-move-trainer-methods

Move ~60 lines of code from subclass to parent abstract class, configure GH actions and backtrack linting

Author: wli51

.github/workflows/pre-commit.yml

@@ -0,0 +1,30 @@
+name: pre-commit
+
+on:
+  pull_request:
+  push:
+    branches: [main]
+jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.11"
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install pre-commit
+
+      - name: Run pre-commit
+        uses: pre-commit/[email protected]
+
+      - name: pre-commit-ci-lite
+        uses: pre-commit-ci/[email protected]
+        if: always()

.pre-commit-config.yaml

@@ -3,13 +3,30 @@ repos:
     rev: v6.0.0
     hooks:
       - id: end-of-file-fixer
+        exclude: |
+          (?x)^(
+            examples/nbconverted/.*|
+            examples/nbconvert/.*|
+            .*\.ipynb$
+          )
 
--   repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.14.0
-    hooks:
-      - id: ruff
-        args: ["--line-length=100"]
-      # linter.
-      - id: ruff-check
-        types_or: [ python, pyi ]
-        args: [ --fix ]
+- repo: https://github.com/astral-sh/ruff-pre-commit
+  rev: v0.14.0
+  hooks:
+    - id: ruff
+      types_or: [python, pyi]
+      args: [--fix, --line-length=100]
+      # Only lint src/ and tests/
+      files: |
+        (?x)^(
+          src/|
+          tests/
+        )
+      # and ignore ipynb-generated Python files if needed:
+      exclude: |
+        (?x)^(
+          examples/.*|
+          .*nbconvert.*|
+          .*nbconverted.*|
+          .*\.ipynb$
+        )