repo-review 0.11.2__tar.gz → 0.12.0__tar.gz

{repo_review-0.11.2 → repo_review-0.12.0}/.github/CONTRIBUTING.md

@@ -32,23 +32,17 @@ $ hatch run example:repo-review <args> # Run an example
 Hatch handles everything for you, including setting up an temporary virtual
 environment.
 
+Using `uv run` directly is also supported.
+
 ## Setting up a development environment manually
 
-You can set up a development environment by running:
+You can set up a development environment in `.venv` by running:
 
 ```bash
-python3 -m venv .venv
-source ./.venv/bin/activate
-pip install -v -e .[dev]
+uv sync
 ```
 
-If you have the [Python Launcher for Unix](https://github.com/brettcannon/python-launcher),
-you can instead do:
-
-```bash
-py -m venv .venv
-py -m install -v -e .[dev]
-```
+Or just prefix every command by `uv run`.
 
 # Post setup
 

{repo_review-0.11.2 → repo_review-0.12.0}/.github/workflows/cd.yml

@@ -6,6 +6,8 @@ on:
     types:
       - published
 
+permissions: {}
+
 jobs:
   dist:
     name: Distribution build
@@ -15,6 +17,7 @@ jobs:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 0
+          persist-credentials: false
 
       - uses: hynek/build-and-inspect-python-package@v2
 

{repo_review-0.11.2 → repo_review-0.12.0}/.github/workflows/ci.yml

@@ -17,12 +17,16 @@ concurrency:
 env:
   FORCE_COLOR: 3
 
+permissions: {}
+
 jobs:
   pre-commit:
     name: Format
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+        with:
+          persist-credentials: false
       - uses: actions/setup-python@v5
         with:
           python-version: "3.x"
@@ -42,6 +46,7 @@ jobs:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 0
+          persist-credentials: false
 
       - uses: actions/setup-python@v5
         with:
@@ -69,6 +74,7 @@ jobs:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 0
+          persist-credentials: false
 
       - uses: hynek/build-and-inspect-python-package@v2
 
@@ -79,6 +85,7 @@ jobs:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 0
+          persist-credentials: false
 
       - name: Setup uv
         uses: yezz123/setup-uv@v4
@@ -108,8 +115,9 @@ jobs:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 0
+          persist-credentials: false
 
       - name: Run repo-review action
         uses: ./
         with:
-          plugins: sp-repo-review==2024.08.19
+          plugins: sp-repo-review==2025.01.22

{repo_review-0.11.2 → repo_review-0.12.0}/.pre-commit-config.yaml

@@ -5,26 +5,26 @@ ci:
 
 repos:
   - repo: https://github.com/adamchainz/blacken-docs
-    rev: 1.18.0
+    rev: 1.19.1
     hooks:
       - id: blacken-docs
-        additional_dependencies: [black==23.*]
+        additional_dependencies: [black==24.*]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.6.1"
+    rev: "v0.9.4"
     hooks:
       - id: ruff
         args: ["--fix", "--show-fixes"]
       - id: ruff-format
 
   - repo: https://github.com/rbubley/mirrors-prettier
-    rev: "v3.3.3"
+    rev: "v3.4.2"
     hooks:
       - id: prettier
         types_or: [yaml, markdown, html, css, scss, javascript, json]
 
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
       - id: check-added-large-files
       - id: check-case-conflict
@@ -45,7 +45,7 @@ repos:
       - id: rst-inline-touching-normal
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.11.1
+    rev: v1.14.1
     hooks:
       - id: mypy
         files: (src|web|tests)
@@ -53,13 +53,14 @@ repos:
         additional_dependencies:
           - click>=8.1.5
           - markdown-it-py
+          - orjson
           - pytest
           - rich
           - tomli
           - types-PyYAML
 
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.3.0
+    rev: v2.4.1
     hooks:
       - id: codespell
         args: ["-Lhist,absense", "-w"]
@@ -78,12 +79,12 @@ repos:
         exclude: .pre-commit-config.yaml
 
   - repo: https://github.com/henryiii/validate-pyproject-schema-store
-    rev: 2024.08.19
+    rev: 2025.02.03
     hooks:
       - id: validate-pyproject
 
   - repo: https://github.com/python-jsonschema/check-jsonschema
-    rev: 0.29.1
+    rev: 0.31.1
     hooks:
       - id: check-dependabot
       - id: check-github-workflows

{repo_review-0.11.2 → repo_review-0.12.0}/.readthedocs.yaml

@@ -13,6 +13,4 @@ build:
     - asdf plugin add uv
     - asdf install uv latest
     - asdf global uv latest
-    - uv venv
-    - uv pip install .[docs]
-    - . .venv/bin/activate && sphinx-build -T -b html -d docs/_build/doctrees -D language=en docs $READTHEDOCS_OUTPUT/html
+    - uv run --group docs sphinx-build -T -b html -d docs/_build/doctrees -D language=en docs $READTHEDOCS_OUTPUT/html

{repo_review-0.11.2 → repo_review-0.12.0}/PKG-INFO

@@ -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 → repo_review-0.12.0}/README.md

@@ -64,8 +64,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
 
@@ -98,7 +102,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
@@ -134,7 +138,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 → repo_review-0.12.0}/docs/checks.md

@@ -30,7 +30,7 @@ explanation instead of the `check()` docstring, you can return a non-empty
 string from the check instead of `False`. Returning `None` makes a check
 "skipped". Docstrings/error messages can access their own object with `{self}`
 and check name with `{name}` (these are processed with `.format()`, so escape `{}`
-as `{{}}`). The error message is in markdown format.
+as `{{}}`). The error message is in Markdown format.
 
 ```{versionchanged} 0.9
 The string return value is not processed via `.format`. You can use `self` and

{repo_review-0.11.2 → repo_review-0.12.0}/docs/cli.md

@@ -17,7 +17,7 @@ pass `--package-dir <path>`.
 
 There are four output formats; `rich` produces great terminal output, `svg`
 produces an SVG based on the rich output, `html` produces a custom HTML report,
-and `json` produces a output that can be processed easily. To make it easier to
+and `json` produces an output that can be processed easily. To make it easier to
 support tools like GitHub Actions, there is also a `--stderr FORMAT` output
 option that produces the selected format on stderr as well, and disables
 producing terminal escape codes on stdout, even if `FORCE_COLOR` is set. This

{repo_review-0.11.2 → repo_review-0.12.0}/docs/families.md

@@ -15,7 +15,7 @@ class Family(typing.TypedDict, total=False):
 
 The `name` will be shown instead if given. The families will be sorted by
 `order` then key. And a `description` will be shown after the name if provided;
-it is expected to be in markdown format.
+it is expected to be in Markdown format.
 
 ```{versionadded} 0.9
 Descriptions are now supported.
@@ -24,7 +24,7 @@ Descriptions are now supported.
 Then you can provide a function that maps family strings to this extra information:
 
 ```python
-def get_familes() -> dict[str, Family]:
+def get_families() -> dict[str, Family]:
     return {
         "general": Family(
             name="General",

{repo_review-0.11.2 → repo_review-0.12.0}/docs/index.html

@@ -6,7 +6,7 @@
       content="initial-scale=1, width=device-width"
     />
     <script
-      src="https://cdn.jsdelivr.net/pyodide/v0.26.2/full/pyodide.js"
+      src="https://cdn.jsdelivr.net/pyodide/v0.27.1/full/pyodide.js"
       crossorigin
     ></script>
     <!-- Production -->
@@ -64,10 +64,10 @@
         <App
           header={true}
           deps={[
-            "repo-review~=0.11.0",
-            "sp-repo-review==2024.08.19",
-            "validate-pyproject-schema-store== 2024.08.19",
-            "validate-pyproject[all]~=0.19.0",
+            "repo-review~=0.11.3",
+            "sp-repo-review==2025.01.22",
+            "validate-pyproject-schema-store==2025.01.20",
+            "validate-pyproject[all]~=0.22.0",
           ]}
         />,
       );

{repo_review-0.11.2 → repo_review-0.12.0}/docs/intro.md

@@ -15,7 +15,7 @@ to use it is:
 pipx run <plugin-name>[cli] .
 ```
 
-This uses [pipx][] (pip for executables) to download the plugin and all of it's
+This uses [pipx][] (pip for executables) to download the plugin and all of its
 dependencies (including repo-review itself) into a temporary virtual
 environment (cached for a week), then runs it. For example:
 
@@ -51,10 +51,26 @@ You can explicitly list checks to select or skip in your `pyproject.toml`:
 
 ```toml
 [tool.repo-review]
-select = ["..."]
-ignore = ["..."]
+select = ["A", "B", "C100"]
+ignore = ["A100"]
 ```
 
+You can list the letter prefix or the exact check name. The ignore list can also
+be a table, with reasons for values. These will be shown explicitly in the report if
+a reason is given.
+
+```toml
+[tool.repo-review.ignore]
+A = "Skipping this whole family"
+B101 = "Skipping this specific check"
+C101 = ""  # Hidden from report, like a normal ignore
+```
+
+If `--select` or `--ignore` are given on the command line, they will override
+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.
+
 ## Pre-commit
 
 You can also use this from pre-commit:

{repo_review-0.11.2 → repo_review-0.12.0}/pyproject.toml

@@ -37,19 +37,44 @@ dependencies = [
   "typing-extensions; python_version < '3.11'",
 ]
 
+[project.urls]
+Changelog = "https://github.com/scientific-python/repo-review/releases"
+Demo = "https://scientific-python.github.io/repo-review"
+Documentation = "https://repo-review.readthedocs.io"
+Homepage = "https://repo-review.readthedocs.io"
+Source = "https://github.com/scientific-python/repo-review"
+
+[project.scripts]
+repo-review = "repo_review.__main__:main"
+
+[project.entry-points."repo_review.fixtures"]
+pyproject = "repo_review.fixtures:pyproject"
+list_all = "repo_review.fixtures:list_all"
+
+[project.entry-points."validate_pyproject.tool_schema"]
+repo-review = "repo_review.schema:get_schema"
+
 [project.optional-dependencies]
 cli = [
   "click >=8",
   "rich >=12.2",
   "rich-click",
 ]
+
+[dependency-groups]
 test = [
   "pytest >=7",
-  "sp-repo-review >=2024.08.19",
+  "sp-repo-review >=2025.01.22",
   "validate-pyproject >=0.14",
 ]
+cov = [
+  { include-group = "test" },
+  "pytest-cov",
+]
 dev = [
-  "repo-review[test,cli]",
+  { include-group = "cov" },
+  "sp-repo-review[cli]",
+  "validate-pyproject-schema-store[all]",
 ]
 docs = [
   "furo",
@@ -63,24 +88,6 @@ docs = [
   "sphinx-github-changelog",
 ]
 
-[project.urls]
-Changelog = "https://github.com/scientific-python/repo-review/releases"
-Demo = "https://scientific-python.github.io/repo-review"
-Documentation = "https://repo-review.readthedocs.io"
-Homepage = "https://repo-review.readthedocs.io"
-Source = "https://github.com/scientific-python/repo-review"
-
-[project.scripts]
-repo-review = "repo_review.__main__:main"
-
-[project.entry-points."repo_review.fixtures"]
-pyproject = "repo_review.fixtures:pyproject"
-list_all = "repo_review.fixtures:list_all"
-
-[project.entry-points."validate_pyproject.tool_schema"]
-repo-review = "repo_review.schema:get_schema"
-
-
 [tool.hatch]
 version.source = "vcs"
 build.hooks.vcs.version-file = "src/repo_review/_version.py"
@@ -89,7 +96,13 @@ build.hooks.vcs.version-file = "src/repo_review/_version.py"
 installer = "uv"
 
 [tool.hatch.envs.hatch-test]
-features = ["test", "cli"]
+features = ["cli"]
+# duplicated since hatch doesn't support groups yet
+dependencies = [
+  "pytest >=7",
+  "sp-repo-review >=2025.01.22",
+  "validate-pyproject >=0.14",
+]
 env-vars.PYTHONWARNDEFAULTENCODING = "1"
 
 [tool.hatch.envs.lint]
@@ -103,8 +116,19 @@ dependencies = ["pylint>=3.2"]
 scripts.lint = "pylint repo_review {args}"
 
 [tool.hatch.envs.docs]
-features = ["docs"]
-dependencies = ["sphinx-autobuild"]
+# duplicated since hatch doesn't support groups yet
+dependencies = [
+  "furo",
+  "myst_parser >=0.13",
+  "repo-review[cli]",
+  "sphinx >=4.0",
+  "sphinx-autodoc-typehints",
+  "sphinx-copybutton",
+  "sphinxcontrib-programoutput",
+  "sphinxext-opengraph",
+  "sphinx-github-changelog",
+  "sphinx-autobuild",
+]
 scripts.linkcheck = "sphinx-build -b=linkcheck docs docs/_build/linkcheck {args}"
 scripts.html = "sphinx-build --keep-going -n -T -b=html docs docs/_build/html {args}"
 scripts.serve = "sphinx-autobuild -n -T -b=html docs docs/_build/html {args}"
@@ -204,7 +228,6 @@ extend-select = [
 ]
 ignore = [
   "ISC001",  # May collide with formatter
-  "PT004",   # Incorrect check, usefixtures is the correct way to do this
   "PLR09",   # Too many X
   "PLR2004", # Magic value in comparison
 ]
@@ -223,3 +246,12 @@ typing-modules = ["repo_review._compat.typing"]
 [tool.ruff.lint.per-file-ignores]
 "src/repo_review/_compat/**.py" = ["TID251"]
 "src/**/__main__.py" = ["T20"]
+
+[tool.coverage]
+report.exclude_also = [
+  "def __dir__()",
+  "if TYPE_CHECKING:",
+  '\.\.\.',
+]
+report.show_missing = true
+report.skip_empty = true

{repo_review-0.11.2 → repo_review-0.12.0}/src/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-0.11.2 → repo_review-0.12.0}/src/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-0.11.2 → repo_review-0.12.0}/src/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-0.11.2 → repo_review-0.12.0}/src/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-0.11.2 → repo_review-0.12.0}/src/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-0.11.2 → repo_review-0.12.0}/src/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-0.11.2 → repo_review-0.12.0}/src/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-0.11.2 → repo_review-0.12.0}/src/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 → repo_review-0.12.0}/tests/test_self.py

@@ -6,12 +6,14 @@ import repo_review.processor
 
 
 @pytest.fixture(autouse=True)
-def patch_entry_points(local_entry_points: object) -> None:  # noqa: ARG001
+def patch_entry_points(local_entry_points: object) -> None:
     pass
 
 
 def test_pyproject() -> None:
-    families, results = repo_review.processor.process(Path())
+    families, results = repo_review.processor.process(
+        Path(), extend_ignore={"X", "PP303"}
+    )
 
     assert families == {
         "general": {},
@@ -26,7 +28,9 @@ def test_pyproject() -> None:
 
 
 def test_no_pyproject() -> None:
-    families, results = repo_review.processor.process(Path("tests"))
+    families, results = repo_review.processor.process(
+        Path("tests"), extend_ignore={"X", "PP303"}
+    )
 
     assert families == {
         "general": {},
@@ -58,12 +62,13 @@ def test_empty_pyproject() -> None:
             "description": "Has flit_core.buildapi backend",
             "name": "PyProject",
         },
+        "skipped": {},
     }
-    assert len(results) == 9
+    assert len(results) == 12
 
     assert (
         sum(result.result is None for result in results if result.family == "pyproject")
-        == 1
+        == 2
     )
     assert (
         sum(result.result for result in results if isinstance(result.result, bool)) == 6
@@ -72,3 +77,6 @@ def test_empty_pyproject() -> None:
         sum(result.result is None for result in results if result.family == "general")
         == 0
     )
+    assert sum(1 for result in results if result.skip_reason) == 3
+    assert sum(1 for result in results if result.skip_reason == "One skip") == 1
+    assert sum(1 for result in results if result.skip_reason == "Group skip") == 2

{repo_review-0.11.2 → repo_review-0.12.0}/tests/test_utilities/pyproject.py

@@ -142,10 +142,51 @@ class PP302(PyProject):
         return "minversion" in options and float(options["minversion"]) >= 6
 
 
-def repo_review_checks() -> dict[str, PyProject | General]:
-    return {p.__name__: p() for p in PyProject.__subclasses__()} | {
+class PP999(PyProject):
+    "Skipped check (single)"
+
+    @staticmethod
+    def check() -> bool:
+        "Not used"
+        return False
+
+
+class X101:
+    "Skipped check (multi)"
+
+    family = "skipped"
+
+    @staticmethod
+    def check() -> bool:
+        "Not used"
+        return False
+
+
+class X102:
+    "Skipped check (multi)"
+
+    family = "skipped"
+
+    @staticmethod
+    def check() -> bool:
+        "Not used"
+        return False
+
+
+def repo_review_checks(
+    pyproject: dict[str, Any],
+) -> dict[str, PyProject | General | X101 | X102]:
+    ret = {p.__name__: p() for p in PyProject.__subclasses__()} | {
         p.__name__: p() for p in General.__subclasses__()
     }
+    extra_checks = (
+        pyproject.get("tool", {}).get("repo-review-local", {}).get("extra", False)
+    )
+    return (
+        (ret | {"X101": X101()} | {"X102": X102()})
+        if extra_checks
+        else {k: v for k, v in ret.items() if k != "PP999"}
+    )
 
 
 def repo_review_families(pyproject: dict[str, Any]) -> dict[str, dict[str, str]]:

{repo_review-0.11.2 → repo_review-0.12.0}/tests/test_utilities/pyproject.toml

@@ -16,3 +16,11 @@ pyproject = "pyproject:repo_review_checks"
 pyproject = "pyproject:repo_review_families"
 
 [tool.example]
+
+
+[tool.repo-review.ignore]
+"PP999" = "One skip"
+"X" = "Group skip"
+
+[tool.repo-review-local]
+extra = true