repo-review 0.11.2__py3-none-any.whl → 0.12.0__py3-none-any.whl

repo_review/__main__.py

@@ -34,9 +34,9 @@ from .ghpath import GHPath
 from .html import to_html
 from .processor import Result, as_simple_dict, collect_all, process
 
-__all__ = ["main", "Formats", "Show", "Status"]
+__all__ = ["Formats", "Show", "Status", "main"]
 
-CODE_THEME = "default"
+CODE_THEME = "ansi_light"
 
 
 def __dir__() -> list[str]:
@@ -163,6 +163,13 @@ def rich_printer(
             msg.append(rich.text.Text.from_markup(description, style=style))
             if result.result is None:
                 msg.append(" [skipped]", style="yellow bold")
+                if result.skip_reason:
+                    sr_style = "yellow"
+                    msg.append(" (", style=sr_style)
+                    msg.append(
+                        rich.text.Text.from_markup(result.skip_reason, style=sr_style)
+                    )
+                    msg.append(")", style=sr_style)
                 tree.add(msg)
             elif result.result:
                 msg.append(rich.text.Text.from_markup(" :white_check_mark:"))
@@ -315,6 +322,16 @@ def _remote_path_processor(package: Path) -> Path | GHPath:
     help="Ignore a check or checks, comma separated.",
     default="",
 )
+@click.option(
+    "--extend-select",
+    help="Checks to run in addition to the ones selected.",
+    default="",
+)
+@click.option(
+    "--extend-ignore",
+    help="Checks to ignore in addition to the ones ignored.",
+    default="",
+)
 @click.option(
     "--package-dir",
     "-p",
@@ -327,6 +344,8 @@ def main(
     stderr_fmt: Formats | None,
     select: str,
     ignore: str,
+    extend_select: str,
+    extend_ignore: str,
     package_dir: str,
     show: Show,
 ) -> None:
@@ -351,6 +370,8 @@ def main(
             stderr_fmt,
             select,
             ignore,
+            extend_select,
+            extend_ignore,
             package_dir,
             add_header=len(packages) > 1,
             show=show,
@@ -378,6 +399,8 @@ def on_each(
     stderr_fmt: Literal["rich", "json", "html", "svg"] | None,
     select: str,
     ignore: str,
+    extend_select: str,
+    extend_ignore: str,
     package_dir: str,
     *,
     add_header: bool,
@@ -387,6 +410,8 @@ def on_each(
 
     ignore_list = {x.strip() for x in ignore.split(",") if x}
     select_list = {x.strip() for x in select.split(",") if x}
+    extend_ignore_list = {x.strip() for x in extend_ignore.split(",") if x}
+    extend_select_list = {x.strip() for x in extend_select.split(",") if x}
 
     collected = collect_all(package, subdir=package_dir)
     if len(collected.checks) == 0:
@@ -407,7 +432,12 @@ def on_each(
         header = package.name
 
     families, processed = process(
-        base_package, select=select_list, ignore=ignore_list, subdir=package_dir
+        base_package,
+        select=select_list,
+        ignore=ignore_list,
+        extend_select=extend_select_list,
+        extend_ignore=extend_ignore_list,
+        subdir=package_dir,
     )
 
     status: Status = "passed" if processed else "empty"

repo_review/_version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '0.11.2'
-__version_tuple__ = version_tuple = (0, 11, 2)
+__version__ = version = '0.12.0'
+__version_tuple__ = version_tuple = (0, 12, 0)

repo_review/checks.py

@@ -6,7 +6,7 @@ from typing import Any, Protocol
 
 from .fixtures import apply_fixtures
 
-__all__ = ["Check", "collect_checks", "is_allowed", "get_check_url"]
+__all__ = ["Check", "collect_checks", "get_check_url", "is_allowed", "name_matches"]
 
 
 def __dir__() -> list[str]:
@@ -70,6 +70,25 @@ def collect_checks(fixtures: Mapping[str, Any]) -> dict[str, Check]:
     }
 
 
+def name_matches(name: str, selectors: Set[str]) -> str:
+    """
+    Checks if the name is contained in the matchers. The selectors can be the
+    exact name or just the non-number prefix. Returns the selector that matched,
+    or an empty string if no match.
+
+    :param name: The name to check.
+    :param expr: The expression to check against.
+
+    :return: The matched selector if the name matches a selector, or an empty string if no match.
+    """
+    if name in selectors:
+        return name
+    short_name = name.rstrip("0123456789")
+    if short_name in selectors:
+        return short_name
+    return ""
+
+
 def is_allowed(select: Set[str], ignore: Set[str], name: str) -> bool:
     """
     Skips the check if the name is in the ignore list or if the name without the
@@ -82,15 +101,10 @@ def is_allowed(select: Set[str], ignore: Set[str], name: str) -> bool:
 
     :return: True if this check is allowed, False otherwise.
     """
-    if (
-        select
-        and name not in select
-        and name.rstrip("0123456789") not in select
-        and "*" not in select
-    ):
+    if select and not name_matches(name, select) and "*" not in select:
         return False
 
-    return name not in ignore and name.rstrip("0123456789") not in ignore
+    return not name_matches(name, ignore)
 
 
 def get_check_url(name: str, check: Check) -> str:

repo_review/fixtures.py

@@ -12,11 +12,11 @@ from ._compat.importlib.resources.abc import Traversable
 from .ghpath import EmptyTraversable
 
 __all__ = [
-    "pyproject",
-    "list_all",
-    "compute_fixtures",
     "apply_fixtures",
     "collect_fixtures",
+    "compute_fixtures",
+    "list_all",
+    "pyproject",
 ]
 
 
@@ -36,8 +36,7 @@ def pyproject(package: Traversable) -> dict[str, Any]:
     pyproject_path = package.joinpath("pyproject.toml")
     if pyproject_path.is_file():
         with pyproject_path.open("rb") as f:
-            # Type ignore fixed in https://github.com/hukkin/tomli/pull/215
-            return tomllib.load(f)  # type: ignore[arg-type]
+            return tomllib.load(f)
     return {}
 
 

repo_review/ghpath.py

@@ -14,7 +14,7 @@ from typing import Literal
 
 from ._compat.importlib.resources.abc import Traversable
 
-__all__ = ["GHPath", "EmptyTraversable"]
+__all__ = ["EmptyTraversable", "GHPath"]
 
 
 def __dir__() -> list[str]:

repo_review/html.py

@@ -61,7 +61,7 @@ def to_html(
                 else "red"
             )
             icon = (
-                "⚠️"
+                ("🔷" if result.skip_reason else "⚠️")
                 if result.result is None
                 else "✅"
                 if result.result
@@ -79,6 +79,11 @@ def to_html(
                 if result.url
                 else result.description
             )
+            if result.skip_reason:
+                description += (
+                    f'<br/><span style="color:DarkKhaki;"><b>Skipped:</b> '
+                    f"<em>{md.render(result.skip_reason)}</em></span>"
+                )
             print(f'<tr style="color: {color};">')
             print(f'<td><span role="img" aria-label="{result_txt}">{icon}</span></td>')
             print(f'<td nowrap="nowrap">{result.name}</td>')

repo_review/processor.py

@@ -17,6 +17,7 @@ from .checks import (
     collect_checks,
     get_check_url,
     is_allowed,
+    name_matches,
     process_result_bool,
 )
 from .families import Family, collect_families
@@ -30,8 +31,8 @@ __all__ = [
     "ResultDict",
     "as_simple_dict",
     "collect_all",
-    "process",
     "md_as_html",
+    "process",
 ]
 
 
@@ -63,6 +64,7 @@ class ResultDict(typing.TypedDict):
     result: bool | None  #: The result, None means skip
     err_msg: str  #: The error message if the result is false, in markdown format
     url: str  #: An optional URL (empty string if missing)
+    skip_reason: str  #: The reason for the skip, if given (empty string if not)
 
 
 @dataclasses.dataclass(frozen=True, kw_only=True)
@@ -75,6 +77,7 @@ class Result:
     name: str  #: The name of the check
     description: str  #: The short description of what the check looks for
     result: bool | None  #: The result, None means skip
+    skip_reason: str = ""  #: The reason for the skip, if given
     err_msg: str = ""  #: The error message if the result is false, in markdown format
     url: str = ""  #: An optional URL (empty string if missing)
 
@@ -179,6 +182,8 @@ def process(
     *,
     select: Set[str] = frozenset(),
     ignore: Set[str] = frozenset(),
+    extend_select: Set[str] = frozenset(),
+    extend_ignore: Set[str] = frozenset(),
     subdir: str = "",
 ) -> ProcessReturn:
     """
@@ -199,8 +204,12 @@ def process(
 
     # Collect our own config
     config = pyproject(package).get("tool", {}).get("repo-review", {})
-    select_checks = select if select else frozenset(config.get("select", ()))
-    skip_checks = ignore if ignore else frozenset(config.get("ignore", ()))
+    ignore_pyproject: list[str] | dict[str, str] = config.get("ignore", [])
+    select_checks = (
+        select if select else frozenset(config.get("select", ()))
+    ) | extend_select
+    skip_checks = (ignore if ignore else frozenset(ignore_pyproject)) | extend_ignore
+    skip_reasons = ignore_pyproject if isinstance(ignore_pyproject, dict) else {}
 
     # Make a graph of the check's interdependencies
     graph: dict[str, Set[str]] = {
@@ -234,9 +243,14 @@ def process(
         result = None if completed[task_name] is None else not completed[task_name]
         doc = check.__doc__ or ""
         err_msg = completed[task_name] or ""
+        skip_reason = ""
 
         if not is_allowed(select_checks, skip_checks, task_name):
-            continue
+            key = name_matches(task_name, skip_reasons.keys())
+            if not key or not skip_reasons.get(key, ""):
+                continue
+            result = None
+            skip_reason = skip_reasons[key]
 
         result_list.append(
             Result(
@@ -246,6 +260,7 @@ def process(
                 result=result,
                 err_msg=textwrap.dedent(err_msg),
                 url=get_check_url(task_name, check),
+                skip_reason=skip_reason,
             )
         )
 

repo_review/testing.py

@@ -13,7 +13,7 @@ from .checks import Check, get_check_url, process_result_bool
 from .fixtures import apply_fixtures
 from .processor import Result
 
-__all__ = ["toml_loads", "compute_check"]
+__all__ = ["compute_check", "toml_loads"]
 
 
 def __dir__() -> list[str]:

{repo_review-0.11.2.dist-info → repo_review-0.12.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
-Metadata-Version: 2.3
+Metadata-Version: 2.4
 Name: repo_review
-Version: 0.11.2
+Version: 0.12.0
 Summary: Framework that can run checks on repos
 Project-URL: Changelog, https://github.com/scientific-python/repo-review/releases
 Project-URL: Demo, https://scientific-python.github.io/repo-review
@@ -34,29 +34,6 @@ Provides-Extra: cli
 Requires-Dist: click>=8; extra == 'cli'
 Requires-Dist: rich-click; extra == 'cli'
 Requires-Dist: rich>=12.2; extra == 'cli'
-Provides-Extra: dev
-Requires-Dist: click>=8; extra == 'dev'
-Requires-Dist: pytest>=7; extra == 'dev'
-Requires-Dist: rich-click; extra == 'dev'
-Requires-Dist: rich>=12.2; extra == 'dev'
-Requires-Dist: sp-repo-review>=2024.08.19; extra == 'dev'
-Requires-Dist: validate-pyproject>=0.14; extra == 'dev'
-Provides-Extra: docs
-Requires-Dist: click>=8; extra == 'docs'
-Requires-Dist: furo; extra == 'docs'
-Requires-Dist: myst-parser>=0.13; extra == 'docs'
-Requires-Dist: rich-click; extra == 'docs'
-Requires-Dist: rich>=12.2; extra == 'docs'
-Requires-Dist: sphinx-autodoc-typehints; extra == 'docs'
-Requires-Dist: sphinx-copybutton; extra == 'docs'
-Requires-Dist: sphinx-github-changelog; extra == 'docs'
-Requires-Dist: sphinx>=4.0; extra == 'docs'
-Requires-Dist: sphinxcontrib-programoutput; extra == 'docs'
-Requires-Dist: sphinxext-opengraph; extra == 'docs'
-Provides-Extra: test
-Requires-Dist: pytest>=7; extra == 'test'
-Requires-Dist: sp-repo-review>=2024.08.19; extra == 'test'
-Requires-Dist: validate-pyproject>=0.14; extra == 'test'
 Description-Content-Type: text/markdown
 
 # repo-review
@@ -125,8 +102,12 @@ select = ["A", "B", "C100"]
 ignore = ["A100"]
 ```
 
+The ignore list can also be a table, with reasons for values.
+
 If `--select` or `--ignore` are given on the command line, they will override
-the `pyproject.toml` config.
+the `pyproject.toml` config. You can use `--extend-select` and `--extend-ignore`
+on the command line to extend the `pyproject.toml` config. These CLI options
+are comma separated.
 
 ## Comparison to other frameworks
 
@@ -159,7 +140,7 @@ webapp. It also would allow `zipfile.Path` to work just as well, too - no need
 to extract.
 
 [Checks][] can request [fixtures][] (like [pytest][]) as arguments. Check files
-can add new fixtures as needed. Fixtures are are specified with entry points,
+can add new fixtures as needed. Fixtures are specified with entry points,
 and take any other fixture as arguments as well - the `root` and `package`
 fixtures represents the root of the repository and of the package you are
 checking, respectively, and are the basis for the other fixtures, which are
@@ -195,7 +176,7 @@ collection functions, the family entry-point also supports fixtures.
 This project inspired [Try-PyHF](https://kratsg.github.io/try-pyhf/), an
 interface for a High Energy Physics package in Scikit-HEP.
 
-This project inspired [abSENSE](https://princetonuniversity.github.io/abSENSE/), an
+This project inspired [abSENSE](https://princetonuniversity.github.io/abSENSE/), a
 web interface to abSENSE.
 
 This was developed for [Scikit-HEP][] before moving to Scientific-Python.

{repo_review-0.11.2.dist-info → repo_review-0.12.0.dist-info}/RECORD

@@ -1,16 +1,16 @@
 repo_review/__init__.py,sha256=U03wTpj7PjSsIROp8O4VRG_NVG-plkx14d_MvqpmJ_w,229
-repo_review/__main__.py,sha256=FjYamqrS55TYqFvz8TuZdJ5-C-l2QP3_6n8lLH04_8o,13890
-repo_review/_version.py,sha256=v52WNNFq-_uq9OSfK9O7vLpi66zlufcyohjNPzWbpdk,413
+repo_review/__main__.py,sha256=-4cUtsnRua2Cwl5ESuBEzXljxWUY2kaAj3kZCFAfwh4,14863
+repo_review/_version.py,sha256=HJyq47ePtlF3WPYAhFFp5LPsrHyF6oju_vSEfklwWro,413
 repo_review/_version.pyi,sha256=j5kbzfm6lOn8BzASXWjGIA1yT0OlHTWqlbyZ8Si_o0E,118
-repo_review/checks.py,sha256=SzTnMxj756xMb9Rffl9K65o2rUGDYfG6CQCj-pNWQzk,4316
+repo_review/checks.py,sha256=Iz5T1d0wDfBIyaO3f5waHEqhmfP_4-9eNOngJpRnpGo,4845
 repo_review/families.py,sha256=TEMQY3whMj8b3iVlWyVsTa0CAZexXRCFXq5uPIkV6bs,2389
-repo_review/fixtures.py,sha256=AHdUhDU--SOrlhWSymiyECp5LmaCGYNRcZHuPpzFUts,3709
-repo_review/ghpath.py,sha256=YadsOkd2hGOrDffSU3PtQiDLgvrVe1GuMKeDFKXwtLs,6123
-repo_review/html.py,sha256=LwoZUqiWC84qSuhwbzqQtwhWDhE8JiPWWTI4nKXF3nA,3485
-repo_review/processor.py,sha256=txXKyw1VP31sclfQL0yac5mXYTkFoca3t-3yKBvqc5k,8275
+repo_review/fixtures.py,sha256=ypeEj-Ae3_3bUWUE0n-wuOrKlEVFU6rp7zK-UoC_t6E,3607
+repo_review/ghpath.py,sha256=fkzUD1G_kuzCmVhTH2yAJoJxxYwjAXfjnBwtaj6nqg0,6123
+repo_review/html.py,sha256=XGfWXNkYK4jQT3KiQRGC4wnhv-DLG-JKttSkqVr7uIM,3759
+repo_review/processor.py,sha256=-p70iGbqwy6Tw-FxxIAXUlgXQxk74fvLQXmojunqikU,8984
 repo_review/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 repo_review/schema.py,sha256=U_iox0khN1oEHCCB5ILffjRvZLG7BR-aaip9QEpMkJk,593
-repo_review/testing.py,sha256=qbjevlG_DQbL4-l-7wTM9rqoLvRaxf7GF66D8i723OY,1915
+repo_review/testing.py,sha256=tA0c-baQ2-0UQMKnY59NgBuLmBqsVcFUBP4uYsL3n6Q,1915
 repo_review/_compat/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 repo_review/_compat/tomllib.py,sha256=caR8Mj-hSBLE2zqdLLjzwOdCBOfikVqS8pWkc4K5vgc,233
 repo_review/_compat/typing.py,sha256=vXfYiJHUaqMShRMbJi0VUg2V59UV-oxceZKSBZ3tfts,246
@@ -19,8 +19,8 @@ repo_review/_compat/importlib/resources/__init__.py,sha256=47DEQpj8HBSa-_TImW-5J
 repo_review/_compat/importlib/resources/abc.py,sha256=KLv7AqUaY-ezZ32lA5yVcZLfMqY6VEo8Q-1jyWpGRbk,300
 repo_review/resources/__init__.py,sha256=RBBaUp-hQrIYBqbAT0MytCXEjDmqVsCjiEgdK_K3dN4,138
 repo_review/resources/repo-review.schema.json,sha256=jAc7ZQV-Hc16ENrNFZm7nlIjiRQGTanq8EjBICkJKpE,791
-repo_review-0.11.2.dist-info/METADATA,sha256=uCkBoPbRRxwzV-ZqOj070MBQiZL5sh4E3l7ENJhgs9M,11437
-repo_review-0.11.2.dist-info/WHEEL,sha256=1yFddiXMmvYK7QYTqtRNtX66WJ0Mz8PYEiEUoOUUxRY,87
-repo_review-0.11.2.dist-info/entry_points.txt,sha256=9XaDWanm31NNpXOpjZh2HxSXR5owVdYJ2cbNG5D8wY8,244
-repo_review-0.11.2.dist-info/licenses/LICENSE,sha256=X7yOxzyAEckYl17p01lZzbraqDVmWEWXE8BnKKRrh7c,1525
-repo_review-0.11.2.dist-info/RECORD,,
+repo_review-0.12.0.dist-info/METADATA,sha256=duKtaL0RIK1pxVs5iH8HQZ3vivRwDjzCKdV4SWJPqH8,10613
+repo_review-0.12.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+repo_review-0.12.0.dist-info/entry_points.txt,sha256=9XaDWanm31NNpXOpjZh2HxSXR5owVdYJ2cbNG5D8wY8,244
+repo_review-0.12.0.dist-info/licenses/LICENSE,sha256=X7yOxzyAEckYl17p01lZzbraqDVmWEWXE8BnKKRrh7c,1525
+repo_review-0.12.0.dist-info/RECORD,,

{repo_review-0.11.2.dist-info → repo_review-0.12.0.dist-info}/WHEEL

@@ -1,4 +1,4 @@
 Wheel-Version: 1.0
-Generator: hatchling 1.25.0
+Generator: hatchling 1.27.0
 Root-Is-Purelib: true
 Tag: py3-none-any