fix: pc check skipped incorrectly (#767)

Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>

Author: henryiii

src/sp_repo_review/checks/precommit.py

@@ -218,7 +218,7 @@ def check(precommit: dict[str, Any], dependabot: dict[str, Any]) -> bool | None:
         ```
         """
         if any(
-            ecosystem.get("package-ecosystem", "") == "github-actions"
+            ecosystem.get("package-ecosystem", "") == "pre-commit"
             for ecosystem in dependabot.get("updates", [])
         ):
             return None
@@ -240,7 +240,7 @@ def check(precommit: dict[str, Any], dependabot: dict[str, Any]) -> bool | None:
         ```
         """
         if any(
-            ecosystem.get("package-ecosystem", "") == "github-actions"
+            ecosystem.get("package-ecosystem", "") == "pre-commit"
             for ecosystem in dependabot.get("updates", [])
         ):
             return None
@@ -262,7 +262,7 @@ def check(precommit: dict[str, Any], dependabot: dict[str, Any]) -> bool | None:
         ```
         """
         if any(
-            ecosystem.get("package-ecosystem", "") == "github-actions"
+            ecosystem.get("package-ecosystem", "") == "pre-commit"
             for ecosystem in dependabot.get("updates", [])
         ):
             return None

tests/test_precommit.py

@@ -290,7 +290,7 @@ def test_pc901():
 def test_pc901_not_needed():
     dependabot = yaml.safe_load("""
         updates:
-          - package-ecosystem: "github-actions"
+          - package-ecosystem: "pre-commit"
     """)
     assert compute_check("PC901", precommit={}, dependabot=dependabot).result is None
 
@@ -324,7 +324,7 @@ def test_pc902_no_msg():
 def test_pc902_not_needed():
     dependabot = yaml.safe_load("""
         updates:
-          - package-ecosystem: "github-actions"
+          - package-ecosystem: "pre-commit"
     """)
     assert compute_check("PC902", precommit={}, dependabot=dependabot).result is None
 
@@ -350,13 +350,13 @@ def test_pc903_no_msg():
 def test_pc903_not_needed():
     dependabot = yaml.safe_load("""
         updates:
-          - package-ecosystem: "github-actions"
+          - package-ecosystem: "pre-commit"
     """)
     assert compute_check("PC903", precommit={}, dependabot=dependabot).result is None
 
 
 def test_repo_review_checks_skips_with_lefthook_only(tmp_path: Path) -> None:
-    """PreCommit checks should be omitted if only lefthook.yml is present.
+    """Precommit checks should be omitted if only lefthook.yml is present.
 
     When a repository uses `lefthook.yml` and does not have a
     `.pre-commit-config.yaml`, `repo_review_checks` should return an empty