From 35d225d95b93119c9d192585f8b229be64ea87c7 Mon Sep 17 00:00:00 2001 From: Henry Schreiner Date: Thu, 16 Nov 2023 15:18:57 -0500 Subject: [PATCH] fix: loosen required summary flag Signed-off-by: Henry Schreiner --- src/sp_repo_review/checks/pyproject.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/sp_repo_review/checks/pyproject.py b/src/sp_repo_review/checks/pyproject.py index 003976c7..4bc1f5ff 100644 --- a/src/sp_repo_review/checks/pyproject.py +++ b/src/sp_repo_review/checks/pyproject.py @@ -205,7 +205,8 @@ class PP308(PyProject): @staticmethod def check(pyproject: dict[str, Any]) -> bool: """ - `-ra` should be in `addopts = [...]` (print summary of all fails/errors). + An explicit summary flag like `-ra` should be in `addopts = [...]` + (print summary of all fails/errors). ```toml [tool.pytest.ini_options] @@ -213,7 +214,7 @@ def check(pyproject: dict[str, Any]) -> bool: ``` """ options = pyproject["tool"]["pytest"]["ini_options"] - return "-ra" in options.get("addopts", []) + return any(opt.startswith("-r") for opt in options.get("addopts", [])) class PP309(PyProject):