run pylint AND ruff

e-valuation · Oct 23, 2023 · 6b19d4f · 6b19d4f
1 parent 493cd34
commit 6b19d4f
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -88,8 +88,10 @@ jobs:
       - name: Setup python
         uses: ./.github/setup_python
 
-      - name: Run linter
-        run: ruff evap
+      - name: Run linters
+        run: |
+          ruff evap
+          pylint evap -j0
 
 
   formatter:

diff --git a/evap/evaluation/management/commands/lint.py b/evap/evaluation/management/commands/lint.py
@@ -9,4 +9,5 @@ class Command(BaseCommand):
     requires_migrations_checks = False
 
     def handle(self, *args, **options):
-        subprocess.run(["ruff", "evap/"], check=False)  # nosec
+        subprocess.run(["pylint", "evap"], check=False)  # nosec
+        subprocess.run(["ruff", "evap"], check=False)  # nosec
diff --git a/evap/evaluation/tests/test_commands.py b/evap/evaluation/tests/test_commands.py
@@ -371,6 +371,7 @@ class TestLintCommand(TestCase):
     def test_pylint_called(mock_subprocess_run):
         management.call_command("lint")
         mock_subprocess_run.assert_called_once_with(["pylint", "evap"], check=False)
+        mock_subprocess_run.assert_called_once_with(["ruff", "evap"], check=False)
 
 
 class TestFormatCommand(TestCase):