Merge pull request #3 from neubig/refactor/use-precommit-for-linting

Refactor linting workflow to use pre-commit

Author: neubig

.github/workflows/lint.yml

@@ -15,7 +15,5 @@ jobs:
       run: |
         python -m pip install --upgrade pip
         pip install ".[dev]"
-    - name: Run ruff
-      run: ruff check .
-    - name: Run mypy
-      run: mypy starter_repo tests
+    - name: Run pre-commit
+      run: pre-commit run --all-files

.pre-commit-config.yaml

@@ -1,11 +1,13 @@
 repos:
--   repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: v0.1.6
+-   repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.4.4
     hooks:
     -   id: ruff
         args: [--fix]
     -   id: ruff-format
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.7.1
+    rev: v1.10.0
     hooks:
-    -   id: mypy
+    -   id: mypy
+        files: ^(starter_repo|tests)/
+        additional_dependencies: [pandas-stubs]

README.md

@@ -61,7 +61,7 @@ pre-commit install
 
 GitHub Actions workflows are set up for:
 
-- [Linting](.github/workflows/lint.yml): Runs Ruff and mypy
+- [Linting](.github/workflows/lint.yml): Runs pre-commit hooks (Ruff and mypy)
 - [Testing](.github/workflows/test.yml): Runs pytest on multiple Python versions