fix: allow flake8 as a linter too

Signed-off-by: Henry Schreiner <henryfs@princeton.edu>

Author: henryiii

src/sp_repo_review/checks/precommit.py

@@ -28,7 +28,7 @@ class PreCommit:
     url = mk_url("style")
     renamed: ClassVar[dict[str, str]] = {}
     repos: ClassVar[set[str]] = set()
-    ids: ClassVar[dict[str, str]] = {}
+    ids: ClassVar[dict[str, set[str]]] = {}
 
     @property
     def describe(self) -> str:
@@ -53,7 +53,7 @@ def check(cls, precommit: dict[str, Any]) -> bool | None | str:
             if repo in cls.repos:
                 if cls.ids and repo in cls.ids:
                     if any(
-                        hook.get("id", "") == cls.ids[repo]
+                        hook.get("id", "") in cls.ids[repo]
                         for hook in repo_item.get("hooks", {})
                     ):
                         return True
@@ -83,7 +83,7 @@ class PC110(PreCommit):
     renamed = {
         "https://github.com/psf/black": "https://github.com/psf/black-pre-commit-mirror"
     }
-    ids = {"https://github.com/astral-sh/ruff-pre-commit": "ruff-format"}
+    ids = {"https://github.com/astral-sh/ruff-pre-commit": {"ruff-format"}}
 
 
 class PC111(PreCommit):
@@ -97,12 +97,17 @@ class PC111(PreCommit):
 
 
 class PC190(PreCommit):
-    "Uses Ruff"
+    "Uses a linter (Ruff/Flake8)"
 
-    repos = {"https://github.com/astral-sh/ruff-pre-commit"}
+    repos = {
+        "https://github.com/astral-sh/ruff-pre-commit",
+        "https://github.com/pycqa/flake8",
+    }
     renamed = {
-        "https://github.com/charliermarsh/ruff-pre-commit": "https://github.com/astral-sh/ruff-pre-commit"
+        "https://github.com/charliermarsh/ruff-pre-commit": "https://github.com/astral-sh/ruff-pre-commit",
+        "https://gitlab.com/pycqa/flake8 ": "https://github.com/pycqa/flake8",
     }
+    ids = {"https://github.com/astral-sh/ruff-pre-commit": {"ruff-check", "ruff"}}
 
 
 class PC140(PreCommit):

tests/test_precommit.py

@@ -76,24 +76,56 @@ def test_pc111_rename():
     assert "adamchainz" in res.err_msg
 
 
-def test_pc190():
+def test_pc190_ruff_check():
     precommit = yaml.safe_load("""
         repos:
           - repo: https://github.com/astral-sh/ruff-pre-commit
+            hooks:
+              - id: ruff-check
+    """)
+    assert compute_check("PC190", precommit=precommit).result
+
+
+def test_pc190_ruff():
+    precommit = yaml.safe_load("""
+        repos:
+          - repo: https://github.com/astral-sh/ruff-pre-commit
+            hooks:
+              - id: ruff
+    """)
+    assert compute_check("PC190", precommit=precommit).result
+
+
+def test_pc190_flake8():
+    precommit = yaml.safe_load("""
+        repos:
+          - repo: https://github.com/pycqa/flake8
     """)
     assert compute_check("PC190", precommit=precommit).result
 
 
-def test_pc190_rename():
+def test_pc190_rename_ruff():
     precommit = yaml.safe_load("""
         repos:
           - repo: https://github.com/charliermarsh/ruff-pre-commit
+            hooks:
+              - id: ruff-check
     """)
     res = compute_check("PC190", precommit=precommit)
     assert not res.result
     assert "astral-sh" in res.err_msg
 
 
+def test_pc190_rename_flake8():
+    precommit = yaml.safe_load("""
+        repos:
+          - repo: https://gitlab.com/pycqa/flake8
+    """)
+    res = compute_check("PC190", precommit=precommit)
+    assert not res.result
+    assert "github" in res.err_msg
+
+
 def test_pc140():
     precommit = yaml.safe_load("""
         repos: