repo-review 0.10.6__tar.gz → 0.11.1__tar.gz

{repo_review-0.10.6 → repo_review-0.11.1}/.devcontainer/devcontainer.json

@@ -1,7 +1,7 @@
 {
   "name": "repo-review[dev]",
-  "image": "mcr.microsoft.com/devcontainers/python:0-3.11",
-  "postCreateCommand": "pipx install nox && pip install -e.[cli,test] pylint",
+  "image": "mcr.microsoft.com/devcontainers/python:0-3.12",
+  "postCreateCommand": "pipx install hatch && pip install -e.[cli,test] pylint",
   "customizations": {
     "vscode": {
       "extensions": [

{repo_review-0.10.6 → repo_review-0.11.1}/.github/CONTRIBUTING.md

@@ -5,28 +5,32 @@ detailed description of best practices for developing Scikit-HEP packages.
 
 ## Quick development
 
-The fastest way to start with development is to use nox. If you don't have nox,
-you can use `pipx run nox` to run it without installing, or `pipx install nox`.
-If you don't have pipx (pip for applications), then you can install with with
-`pip install pipx` (the only case were installing an application with regular
-pip is reasonable). If you use macOS, then pipx and nox are both in brew, use
-`brew install pipx nox`.
+The fastest way to start with development is to use hatch. If you don't have
+hatch, you can use `pipx run hatch` to run it without installing, or `pipx
+install hatch`. If you don't have pipx (pip for applications), then you can
+install with with `pip install pipx` (the only case were installing an
+application with regular pip is reasonable). If you use macOS, then pipx and
+hatch are both in brew, use `brew install pipx hatch`. Hatch 1.10+ is required.
 
-To use, run `nox`. This will lint and test using every installed version of
-Python on your system, skipping ones that are not installed. You can also run
-specific jobs:
+Here are some common tasks you might want to run:
 
 ```console
-$ nox -s lint  # Lint only
-$ nox -s tests # Tests
-$ nox -s docs -- --serve  # Build and serve the docs
-$ nox -s build  # Make an SDist and wheel
+$ hatch run lint:lint        # all linters (pre-commit)
+$ hatch run pylint:lint      # pylint
+$ hatch fmt                  # just format & basic lint
+$ hatch tests                # run the tests
+$ hatch build                # build SDist and wheel
+$ hatch run docs:serve       # build and serve the docs
+$ hatch run docs:html        # just build the docs
+$ hatch run docs:man         # build manpage
+$ hatch run docs:linkcheck   # check for broken links
+$ hatch run api-docs:build   # rebuild the API docs
+$ hatch run webapp:serve     # serve the webapp
+$ hatch run example:repo-review <args> # Run an example
 ```
 
-Nox handles everything for you, including setting up an temporary virtual
-environment for each run.
-
-You can also use `nox -s run -- .` to run an example set of checks on a repo.
+Hatch handles everything for you, including setting up an temporary virtual
+environment.
 
 ## Setting up a development environment manually
 
@@ -71,19 +75,25 @@ pytest
 You can build the docs using:
 
 ```bash
-nox -s docs
+hatch run docs:docs
 ```
 
 You can see a preview with:
 
 ```bash
-nox -s docs -- --serve
+hatch run docs:serve
+```
+
+You can rebuild the API docs with:
+
+```bash
+$ hatch run api-docs:build
 ```
 
 ## Pre-commit
 
 This project uses pre-commit for all style checking. While you can run it with
-nox, this is such an important tool that it deserves to be installed on its
+hatch, this is such an important tool that it deserves to be installed on its
 own. Install pre-commit and run:
 
 ```bash
@@ -94,5 +104,5 @@ to check all files.
 
 ## Running DevContainer
 
-You can use DevContainer, such as in GitHub Codespaces or locally. Nox and a
+You can use DevContainer, such as in GitHub Codespaces or locally. Hatch and a
 local install will be available.

repo_review-0.11.1/.github/ISSUE_TEMPLATE/new-issue.md

@@ -0,0 +1,11 @@
+---
+name: New Issue
+about: Prepare a new bug report or feature request
+title: ""
+labels: ""
+assignees: ""
+---
+
+<!--
+Thanks for making a bug report or feature request! Please note, repo-review is a framework for running checks. If you have an issue with a particular check, please open the issue with the plugin that provides the check. The repo-review demo uses the sp-repo-review plugin, which lives at http://github.com/scientific-python/cookie, for example.
+-->

repo_review-0.11.1/.github/release.yml

@@ -0,0 +1,5 @@
+changelog:
+  exclude:
+    authors:
+      - dependabot
+      - pre-commit-ci

{repo_review-0.10.6 → repo_review-0.11.1}/.github/workflows/ci.yml

@@ -27,21 +27,15 @@ jobs:
         with:
           python-version: "3.x"
       - uses: pre-commit/action@v3.0.1
-        with:
-          extra_args: --hook-stage manual --all-files
       - name: Run PyLint
-        run: |
-          echo "::add-matcher::$GITHUB_WORKSPACE/.github/matchers/pylint.json"
-          pipx run --python=python nox -s pylint
+        run: pipx run --python=python hatch run pylint:lint --output-format=github
 
   checks:
-    name: Check Python ${{ matrix.python-version }} on ${{ matrix.runs-on }}
+    name: Check Python on ${{ matrix.runs-on }}
     runs-on: ${{ matrix.runs-on }}
-    needs: [pre-commit]
     strategy:
       fail-fast: false
       matrix:
-        python-version: ["3.10", "3.11", "3.12"]
         runs-on: [ubuntu-latest, macos-latest, windows-latest]
 
     steps:
@@ -51,20 +45,21 @@ jobs:
 
       - uses: actions/setup-python@v5
         with:
-          python-version: ${{ matrix.python-version }}
+          python-version: |
+            3.10
+            3.11
+            3.12
+            3.13
+          allow-prereleases: true
 
       - name: Setup uv
         uses: yezz123/setup-uv@v4
-        with:
-          uv-venv: ".venv"
 
-      - name: Install package
-        run: uv pip install .[test,cli]
+      - name: Install hatch
+        run: uv pip install --system hatch
 
-      - name: Test package
-        run: python -m pytest -ra
-        env:
-          PYTHONUTF8: "1"
+      - name: Test package (all Pythons)
+        run: hatch test -a
 
   dist:
     name: Distribution build
@@ -88,19 +83,22 @@ jobs:
       - name: Setup uv
         uses: yezz123/setup-uv@v4
 
-      - uses: wntrblm/nox@2024.04.15
+      - uses: actions/setup-python@v5
         with:
-          python-versions: "3.11"
+          python-version: "3.12"
+
+      - name: Install hatch
+        run: uv pip install --system hatch
 
       - name: Linkcheck
-        run: nox -s docs -- -b linkcheck
+        run: hatch run docs:linkcheck
 
       - name: Build docs with warnings as errors
-        run: nox -s docs -- -W
+        run: hatch run docs:html -W
 
       - name: Verify no changes required to API docs
         run: |
-          nox -s build_api_docs
+          hatch run api-docs:build
           git diff --exit-code
 
   action:
@@ -114,4 +112,4 @@ jobs:
       - name: Run repo-review action
         uses: ./
         with:
-          plugins: sp-repo-review==2024.03.10
+          plugins: sp-repo-review==2024.04.23

{repo_review-0.10.6 → repo_review-0.11.1}/.pre-commit-config.yaml

@@ -1,23 +1,24 @@
 ci:
   autoupdate_commit_msg: "chore(deps): update pre-commit hooks"
   autofix_commit_msg: "style: pre-commit fixes"
+  autoupdate_schedule: monthly
 
 repos:
   - repo: https://github.com/adamchainz/blacken-docs
-    rev: 1.16.0
+    rev: 1.18.0
     hooks:
       - id: blacken-docs
         additional_dependencies: [black==23.*]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.4.1"
+    rev: "v0.6.1"
     hooks:
       - id: ruff
         args: ["--fix", "--show-fixes"]
       - id: ruff-format
 
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v4.0.0-alpha.8"
+  - repo: https://github.com/rbubley/mirrors-prettier
+    rev: "v3.3.3"
     hooks:
       - id: prettier
         types_or: [yaml, markdown, html, css, scss, javascript, json]
@@ -44,7 +45,7 @@ repos:
       - id: rst-inline-touching-normal
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.9.0
+    rev: v1.11.1
     hooks:
       - id: mypy
         files: (src|web|tests)
@@ -58,7 +59,7 @@ repos:
           - types-PyYAML
 
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.2.6
+    rev: v2.3.0
     hooks:
       - id: codespell
         args: ["-Lhist,absense", "-w"]
@@ -75,3 +76,15 @@ repos:
         language: pygrep
         entry: PyBind|Numpy|Cmake|CCache|Github|PyTest
         exclude: .pre-commit-config.yaml
+
+  - repo: https://github.com/henryiii/validate-pyproject-schema-store
+    rev: 2024.08.19
+    hooks:
+      - id: validate-pyproject
+
+  - repo: https://github.com/python-jsonschema/check-jsonschema
+    rev: 0.29.1
+    hooks:
+      - id: check-dependabot
+      - id: check-github-workflows
+      - id: check-readthedocs

repo_review-0.11.1/.readthedocs.yaml

@@ -0,0 +1,18 @@
+# .readthedocs.yaml
+# Read the Docs configuration file
+# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
+
+# Required
+version: 2
+
+build:
+  os: "ubuntu-22.04"
+  tools:
+    python: "3.12"
+  commands:
+    - 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

{repo_review-0.10.6 → repo_review-0.11.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: repo_review
-Version: 0.10.6
+Version: 0.11.1
 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
@@ -21,6 +21,7 @@ Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Topic :: Software Development :: Quality Assurance
 Classifier: Typing :: Typed
@@ -38,7 +39,7 @@ 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>=2023.12.21; extra == 'dev'
+Requires-Dist: sp-repo-review>=2024.04.23; extra == 'dev'
 Requires-Dist: validate-pyproject>=0.14; extra == 'dev'
 Provides-Extra: docs
 Requires-Dist: click>=8; extra == 'docs'
@@ -54,7 +55,7 @@ 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>=2023.12.21; extra == 'test'
+Requires-Dist: sp-repo-review>=2024.04.23; extra == 'test'
 Requires-Dist: validate-pyproject>=0.14; extra == 'test'
 Description-Content-Type: text/markdown
 
@@ -76,7 +77,7 @@ least one plugin to be installed.
 With one or more plugins, it will produce a list of results - green checkmarks
 mean this rule is followed, red x’s mean the rule is not. A yellow warning sign
 means that the check was skipped because a previous required check failed. Four
-output formats are supported; `rich`, `svg`, `html`, and `json`.
+output formats are supported: `rich`, `svg`, `html`, and `json`.
 
 ## Plugins
 
@@ -140,7 +141,7 @@ the files that interest the plugin in question.
 
 So if you want to lint Python code, use Flake8 or Ruff. But if you want to
 check Flake8 or Ruff's configuration, use repo-review! Generally, repo-review
-plugins are more about requiring things to be present, like making use all your
+plugins are more about requiring things to be present, like making sure all your
 repos have some [pre-commit][] check.
 
 ## Development of repo-review and plugins
@@ -169,7 +170,7 @@ accept fixtures, allowing dynamic check listings.
 Check files do not depend on the main library, and can be extended (similar to
 Flake8). You register new check files via entry-points - so extending this is
 with custom checks or custom fixtures is easy and trivial. There's no need to
-subclass or do anything with the base library - no dependency required.
+subclass or do anything with the base library - no dependency on repo-review required.
 
 Checks are as simple as possible so they are easy to write. A check is a class
 with the name (1-2 letters + number) and a docstring (the check message). It

{repo_review-0.10.6 → repo_review-0.11.1}/README.md

@@ -16,7 +16,7 @@ least one plugin to be installed.
 With one or more plugins, it will produce a list of results - green checkmarks
 mean this rule is followed, red x’s mean the rule is not. A yellow warning sign
 means that the check was skipped because a previous required check failed. Four
-output formats are supported; `rich`, `svg`, `html`, and `json`.
+output formats are supported: `rich`, `svg`, `html`, and `json`.
 
 ## Plugins
 
@@ -80,7 +80,7 @@ the files that interest the plugin in question.
 
 So if you want to lint Python code, use Flake8 or Ruff. But if you want to
 check Flake8 or Ruff's configuration, use repo-review! Generally, repo-review
-plugins are more about requiring things to be present, like making use all your
+plugins are more about requiring things to be present, like making sure all your
 repos have some [pre-commit][] check.
 
 ## Development of repo-review and plugins
@@ -109,7 +109,7 @@ accept fixtures, allowing dynamic check listings.
 Check files do not depend on the main library, and can be extended (similar to
 Flake8). You register new check files via entry-points - so extending this is
 with custom checks or custom fixtures is easy and trivial. There's no need to
-subclass or do anything with the base library - no dependency required.
+subclass or do anything with the base library - no dependency on repo-review required.
 
 Checks are as simple as possible so they are easy to write. A check is a class
 with the name (1-2 letters + number) and a docstring (the check message). It

{repo_review-0.10.6 → repo_review-0.11.1}/action.yml

@@ -26,7 +26,7 @@ runs:
     - uses: actions/setup-python@v5
       id: python
       with:
-        python-version: "3.11"
+        python-version: "3.12"
         update-environment: false
 
     - name: Install repo-review and plugins

{repo_review-0.10.6 → repo_review-0.11.1}/docs/conf.py

@@ -1,8 +1,11 @@
+from __future__ import annotations
+
 import importlib
 import importlib.metadata
 import inspect
 import os
 from pathlib import Path
+from typing import Any
 
 import repo_review
 
@@ -39,6 +42,24 @@ exclude_patterns = [
 
 html_theme = "furo"
 
+html_theme_options: dict[str, Any] = {
+    "footer_icons": [
+        {
+            "name": "GitHub",
+            "url": "https://github.com/scientific-python/repo-review",
+            "html": """
+                <svg stroke="currentColor" fill="currentColor" stroke-width="0" viewBox="0 0 16 16">
+                    <path fill-rule="evenodd" d="M8 0C3.58 0 0 3.58 0 8c0 3.54 2.29 6.53 5.47 7.59.4.07.55-.17.55-.38 0-.19-.01-.82-.01-1.49-2.01.37-2.53-.49-2.69-.94-.09-.23-.48-.94-.82-1.13-.28-.15-.68-.52-.01-.53.63-.01 1.08.58 1.23.82.72 1.21 1.87.87 2.33.66.07-.52.28-.87.51-1.07-1.78-.2-3.64-.89-3.64-3.95 0-.87.31-1.59.82-2.15-.08-.2-.36-1.02.08-2.12 0 0 .67-.21 2.2.82.64-.18 1.32-.27 2-.27.68 0 1.36.09 2 .27 1.53-1.04 2.2-.82 2.2-.82.44 1.1.16 1.92.08 2.12.51.56.82 1.27.82 2.15 0 3.07-1.87 3.75-3.65 3.95.29.25.54.73.54 1.48 0 1.07-.01 1.93-.01 2.2 0 .21.15.46.55.38A8.013 8.013 0 0 0 16 8c0-4.42-3.58-8-8-8z"></path>
+                </svg>
+            """,
+            "class": "",
+        },
+    ],
+    "source_repository": "https://github.com/scientific-python/repo-review/",
+    "source_branch": "main",
+    "source_directory": "docs/",
+}
+
 myst_enable_extensions = [
     "colon_fence",
     "deflist",

{repo_review-0.10.6 → repo_review-0.11.1}/docs/index.html

@@ -6,7 +6,7 @@
       content="initial-scale=1, width=device-width"
     />
     <script
-      src="https://cdn.jsdelivr.net/pyodide/v0.23.3/full/pyodide.js"
+      src="https://cdn.jsdelivr.net/pyodide/v0.26.2/full/pyodide.js"
       crossorigin
     ></script>
     <!-- Production -->
@@ -19,7 +19,7 @@
       crossorigin
     ></script>
     <script
-      src="https://unpkg.com/@mui/material@v5.13.3/umd/material-ui.production.min.js"
+      src="https://unpkg.com/@mui/material@v5.16.7/umd/material-ui.production.min.js"
       crossorigin
     ></script>
     <!-- Development
@@ -32,7 +32,7 @@
       crossorigin
     ></script>
     <script
-      src="https://unpkg.com/@mui/material@v5.13.3/umd/material-ui.development.js"
+      src="https://unpkg.com/@mui/material@v5.16.7/umd/material-ui.development.js"
       crossorigin
     ></script>
     -->
@@ -64,10 +64,10 @@
         <App
           header={true}
           deps={[
-            "repo-review~=0.10.0",
-            "sp-repo-review==2024.03.10",
-            "validate-pyproject-schema-store==2024.04.20",
-            "validate-pyproject[all]~=0.16.0",
+            "repo-review~=0.11.0",
+            "sp-repo-review==2024.04.23",
+            "validate-pyproject-schema-store== 2024.08.19",
+            "validate-pyproject[all]~=0.19.0",
           ]}
         />,
       );

{repo_review-0.10.6 → repo_review-0.11.1}/docs/plugins.md

@@ -43,7 +43,7 @@ def test_has_tool_ruff(tmp_path: Path) -> None:
     # In this example, tool.ruff is required to be present by this plugin
     d = tmp_path / "some_package"
     d.mkdir()
-    d.joinpath("pyproject.toml").write_text("[tool.ruff]")
+    d.joinpath("pyproject.toml").write_text("[tool.ruff]", encoding="utf-8")
     processed = repo_review_processor.process(d)
     assert all(r.result for r in processed.results), f"{processed.results}"
 ```

{repo_review-0.10.6 → repo_review-0.11.1}/docs/webapp.js

@@ -33,7 +33,6 @@ function IfUrlLink({ name, url, color }) {
     return (
       <MaterialUI.Typography
         sx={{ display: "inline" }}
-        component="span"
         variant="body2"
         color={color}
         component="a"
@@ -192,12 +191,13 @@ function MyThemeProvider(props) {
 class App extends React.Component {
   constructor(props) {
     super(props);
-    const deps_str = props.deps.join(" ");
+    const inner_deps_str = props.deps.join("\n");
+    const deps_str = `<pre><code>${inner_deps_str}</code></pre>`;
     this.state = {
       results: [],
       repo: urlParams.get("repo") || "",
       branch: urlParams.get("branch") || "",
-      msg: `${DEFAULT_MSG} Packages: ${deps_str}`,
+      msg: `<p>${DEFAULT_MSG}</p><h4>Packages:</h4> ${deps_str}`,
       progress: false,
       err_msg: "",
       url: "",
@@ -229,12 +229,9 @@ class App extends React.Component {
       var families_checks;
       try {
         families_checks = pyodide.runPython(`
-          from pyodide.http import open_url
           from repo_review.processor import process, md_as_html
           from repo_review.ghpath import GHPath
 
-          GHPath.open_url = staticmethod(open_url)
-
           package = GHPath(repo="${state.repo}", branch="${state.branch}")
           result = process(package)
 
@@ -366,7 +363,7 @@ class App extends React.Component {
           <MaterialUI.Paper elevation={3}>
             <MaterialUI.Box sx={{ p: 2 }}>
               <MaterialUI.Typography variant="body1" component="div">
-                {this.state.msg}
+                <span dangerouslySetInnerHTML={{ __html: this.state.msg }} />
               </MaterialUI.Typography>
               {this.state.progress && <MaterialUI.LinearProgress />}
               {this.state.err_msg && (

{repo_review-0.10.6 → repo_review-0.11.1}/docs/webapp.md

@@ -20,7 +20,7 @@ You can also use the `html` output and write your own webapp. You need to provid
 
 ```html
 <script
-  src="https://cdn.jsdelivr.net/pyodide/v0.23.2/full/pyodide.js"
+  src="https://cdn.jsdelivr.net/pyodide/v0.25.1/full/pyodide.js"
   crossorigin
 ></script>
 ```
@@ -44,13 +44,10 @@ You can get the families and the checks:
 
 ```js
 result_html_py = pyodide.runPython(`
-  from pyodide.http import open_url
   from repo_review.processor import process
   from repo_review.ghpath import GHPath
   from repo_review.html import to_html
 
-  GHPath.open_url = staticmethod(open_url)
-
   package = GHPath(repo="${state.repo}", branch="${state.branch}")
   to_html(*process(package))
 `);

{repo_review-0.10.6 → repo_review-0.11.1}/pyproject.toml

@@ -23,6 +23,7 @@ classifiers = [
   "Programming Language :: Python :: 3.10",
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
+  "Programming Language :: Python :: 3.13",
   "Programming Language :: Python",
   "Topic :: Software Development :: Quality Assurance",
   "Topic :: Software Development :: Libraries :: Python Modules",
@@ -44,7 +45,7 @@ cli = [
 ]
 test = [
   "pytest >=7",
-  "sp-repo-review >=2023.12.21",
+  "sp-repo-review >=2024.04.23",
   "validate-pyproject >=0.14",
 ]
 dev = [
@@ -84,6 +85,47 @@ repo-review = "repo_review.schema:get_schema"
 version.source = "vcs"
 build.hooks.vcs.version-file = "src/repo_review/_version.py"
 
+[tool.hatch.envs.default]
+installer = "uv"
+
+[tool.hatch.envs.hatch-test]
+features = ["test", "cli"]
+env-vars.PYTHONWARNDEFAULTENCODING = "1"
+
+[tool.hatch.envs.lint]
+dependencies = ["pre-commit"]
+skip-install = true
+scripts.lint = "pre-commit run --all-files --show-diff-on-failure {args}"
+
+[tool.hatch.envs.pylint]
+features = ["cli"]
+dependencies = ["pylint>=3.2"]
+scripts.lint = "pylint repo_review {args}"
+
+[tool.hatch.envs.docs]
+features = ["docs"]
+dependencies = ["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}"
+scripts.man = "sphinx-build --keep-going -n -T -b=man docs docs/_build/man {args}"
+
+[tool.hatch.envs.api-docs]
+skip-install = true
+dependencies = ["sphinx"]
+scripts.build = "sphinx-apidoc -o docs/api/ --module-first --no-toc --force src/repo_review"
+
+[tool.hatch.envs.example]
+features = ["cli"]
+post-install-commands = ["$HATCH_UV pip install -e tests/test_utilities"]
+
+[tool.hatch.envs.webapp]
+skip-install = true
+scripts.serve = "cd docs && echo 'Serving on http://localhost:8080' && python -m http.server 8080"
+
+[[tool.hatch.envs.hatch-test.matrix]]
+python = ["3.13", "3.12", "3.11", "3.10"]
+
 
 [tool.pytest.ini_options]
 minversion = "7.0"
@@ -92,6 +134,8 @@ xfail_strict = true
 log_cli_level = "INFO"
 filterwarnings = [
   "error",
+  "ignore:OptionHighlighter:DeprecationWarning",  # rich-click triggers its own warning
+  "ignore:`highlighter`:DeprecationWarning",  # rich-click triggers its own warning
 ]
 testpaths = ["tests"]
 pythonpath = ["tests/test_utilities"]
@@ -113,8 +157,9 @@ disallow_untyped_defs = true
 
 
 [tool.pylint]
-master.py-version = "3.10"
-master.ignore-paths= ["src/repo_review/_version.py"]
+py-version = "3.10"
+ignore-paths = ["src/repo_review/_version.py"]
+ignored-modules = ["pyodide"]
 reports.output-format = "colorized"
 similarities.ignore-imports = "yes"
 messages_control.disable = [
@@ -131,12 +176,10 @@ messages_control.disable = [
   "no-member",  # better handled by mypy, etc.
   "used-before-assignment",  # False positive on conditional import
   "unnecessary-ellipsis",  # Not correct for typing
+  "import-outside-toplevel",  # better handled elsewhere
 ]
 
 
-[tool.ruff]
-src = ["src"]
-
 [tool.ruff.lint]
 extend-select = [
   "B",           # flake8-bugbear
@@ -160,7 +203,7 @@ extend-select = [
   "YTT",         # flake8-2020
 ]
 ignore = [
-  "ISC001", # May collide with formatter
+  "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
@@ -181,6 +224,5 @@ typing-modules = ["repo_review._compat.typing"]
 "src/repo_review/_compat/**.py" = ["TID251"]
 "src/**/__main__.py" = ["T20"]
 
-
 [tool.repo-review]
-ignore = ["PC110"]
+ignore = ["RF003", "PC180"]  # Will be fixed in next sp-repo-review release

{repo_review-0.10.6 → repo_review-0.11.1}/src/repo_review/__main__.py

@@ -2,6 +2,7 @@ from __future__ import annotations
 
 import itertools
 import json
+import os
 import sys
 import typing
 from collections.abc import Mapping, Sequence
@@ -83,6 +84,17 @@ def rich_printer(
     status: Status,
     header: str = "",
 ) -> None:
+    # Before Python 3.15, this isn't always unicode
+    if (
+        sys.version_info < (3, 15)
+        and "PYTHONIOENCODING" not in os.environ
+        and "PYTHONUTF8" not in os.environ
+    ):
+        if sys.stdout.encoding != "utf-8":
+            sys.stdout.reconfigure(encoding="utf-8")  # type: ignore[union-attr]
+        if sys.stderr.encoding != "utf-8":
+            sys.stderr.reconfigure(encoding="utf-8")  # type: ignore[union-attr]
+
     console = rich.console.Console(
         record=svg, quiet=svg, stderr=stderr, color_system="auto" if color else None
     )

{repo_review-0.10.6 → repo_review-0.11.1}/src/repo_review/_version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '0.10.6'
-__version_tuple__ = version_tuple = (0, 10, 6)
+__version__ = version = '0.11.1'
+__version_tuple__ = version_tuple = (0, 11, 1)

{repo_review-0.10.6 → repo_review-0.11.1}/src/repo_review/checks.py

@@ -89,9 +89,8 @@ def is_allowed(select: Set[str], ignore: Set[str], name: str) -> bool:
         and "*" not in select
     ):
         return False
-    if name in ignore or name.rstrip("0123456789") in ignore:
-        return False
-    return True
+
+    return name not in ignore and name.rstrip("0123456789") not in ignore
 
 
 def get_check_url(name: str, check: Check) -> str:

{repo_review-0.10.6 → repo_review-0.11.1}/src/repo_review/ghpath.py

@@ -7,6 +7,7 @@ from __future__ import annotations
 import dataclasses
 import io
 import json
+import sys
 import typing
 from collections.abc import Iterator
 from typing import Literal
@@ -47,7 +48,12 @@ class GHPath(Traversable):
 
     @staticmethod
     def open_url(url: str) -> io.StringIO:
-        "This method can be overridden for pyodide with pyodide.open_url"
+        "This method can be overridden manually for WASM. Supports pyodide currently."
+        if sys.platform == "emscripten":
+            import pyodide.http
+
+            return pyodide.http.open_url(url)
+
         import urllib.request  # pylint: disable=import-outside-toplevel
 
         with urllib.request.urlopen(url) as response:

{repo_review-0.10.6 → repo_review-0.11.1}/src/repo_review/resources/repo-review.schema.json

@@ -9,7 +9,18 @@
       "$ref": "#/$defs/checks"
     },
     "ignore": {
-      "$ref": "#/$defs/checks"
+      "oneOf": [
+        {
+          "$ref": "#/$defs/checks"
+        },
+        {
+          "type": "object",
+          "patternProperties": {
+            "^[A-Z]+[0-9]*$": { "type": "string" }
+          },
+          "additionalProperties": false
+        }
+      ]
     }
   },
   "$defs": {

{repo_review-0.10.6 → repo_review-0.11.1}/tests/conftest.py

@@ -9,7 +9,7 @@ def nocolor(monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.delenv("FORCE_COLOR", raising=False)
 
 
-@pytest.fixture()
+@pytest.fixture
 def no_entry_points(monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.setattr(
         importlib.metadata,
@@ -18,7 +18,7 @@ def no_entry_points(monkeypatch: pytest.MonkeyPatch) -> None:
     )
 
 
-@pytest.fixture()
+@pytest.fixture
 def local_entry_points(monkeypatch: pytest.MonkeyPatch) -> None:
     orig_ep = importlib.metadata.entry_points
     ep1 = importlib.metadata.EntryPoint(

{repo_review-0.10.6 → repo_review-0.11.1}/tests/test_multi.py

@@ -10,7 +10,7 @@ from click.testing import CliRunner
 from repo_review.__main__ import main
 
 
-@pytest.fixture()
+@pytest.fixture
 def multiple_packages(tmp_path: Path) -> tuple[str, str]:
     packages = (tmp_path / "package_1", tmp_path / "package_2")
 
@@ -28,7 +28,7 @@ def multiple_packages(tmp_path: Path) -> tuple[str, str]:
             """
         )
 
-        package.joinpath("pyproject.toml").write_text(basic_toml)
+        package.joinpath("pyproject.toml").write_text(basic_toml, encoding="utf-8")
         package.joinpath(".pre-commit-config.yaml").touch()
         package.joinpath("README.md").touch()
 
@@ -41,7 +41,7 @@ def multiple_packages(tmp_path: Path) -> tuple[str, str]:
 @pytest.mark.usefixtures("local_entry_points")
 def test_multiple_packages_rich(multiple_packages: Sequence[str]) -> None:
     runner = CliRunner()
-    result = runner.invoke(main, [*multiple_packages])
+    result = runner.invoke(main, [*multiple_packages], catch_exceptions=False)
     assert result.exit_code == 0
     assert "package_1" in result.output
     assert "package_2" in result.output
@@ -51,7 +51,9 @@ def test_multiple_packages_rich(multiple_packages: Sequence[str]) -> None:
 @pytest.mark.usefixtures("local_entry_points")
 def test_multiple_packages_json(multiple_packages: Sequence[str]) -> None:
     runner = CliRunner()
-    result = runner.invoke(main, [*multiple_packages, "--format", "json"])
+    result = runner.invoke(
+        main, [*multiple_packages, "--format", "json"], catch_exceptions=False
+    )
     assert result.exit_code == 0
     output = json.loads(result.output)
     assert len(output) == 2
@@ -62,7 +64,9 @@ def test_multiple_packages_json(multiple_packages: Sequence[str]) -> None:
 @pytest.mark.usefixtures("local_entry_points")
 def test_multiple_packages_html(multiple_packages: Sequence[str]) -> None:
     runner = CliRunner()
-    result = runner.invoke(main, [*multiple_packages, "--format", "html"])
+    result = runner.invoke(
+        main, [*multiple_packages, "--format", "html"], catch_exceptions=False
+    )
     assert result.exit_code == 0
     tree = ET.fromstring(f"<body>{result.output}</body>")
     assert len(tree) == 2

{repo_review-0.10.6 → repo_review-0.11.1}/tests/test_package.py

@@ -36,16 +36,54 @@ def test_broken_validate_pyproject(tmp_path: Path) -> None:
                 [tool.repo-review]
                 ignore = ["a2"]
             """
-        )
+        ),
+        encoding="utf-8",
     )
 
     results = process(tmp_path)
 
     (result,) = (r for r in results.results if r.name == "VPP001")
-    assert "must match pattern" in result.err_msg
+    assert "must be valid exactly by one definition" in result.err_msg
     assert not result.result
 
 
+def test_broken_validate_pyproject_object(tmp_path: Path) -> None:
+    pytest.importorskip("validate_pyproject")
+    tmp_path.joinpath("pyproject.toml").write_text(
+        textwrap.dedent(
+            """\
+                [tool.repo-review.ignore]
+                a2 = "some message"
+            """
+        ),
+        encoding="utf-8",
+    )
+
+    results = process(tmp_path)
+
+    (result,) = (r for r in results.results if r.name == "VPP001")
+    assert "must be valid exactly by one definition" in result.err_msg
+    assert not result.result
+
+
+def test_working_validate_pyproject_object(tmp_path: Path) -> None:
+    pytest.importorskip("validate_pyproject")
+    tmp_path.joinpath("pyproject.toml").write_text(
+        textwrap.dedent(
+            """\
+                [tool.repo-review.ignore]
+                PP102 = "some message"
+            """
+        ),
+        encoding="utf-8",
+    )
+
+    results = process(tmp_path)
+
+    (result,) = (r for r in results.results if r.name == "VPP001")
+    assert result.result
+
+
 def test_testing_function():
     pytest.importorskip("sp_repo_review")
 

{repo_review-0.10.6 → repo_review-0.11.1}/tests/test_utilities/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "flit_core.buildapi"
 [project]
 name = "pyproject"
 version = "0.0.1"
-classfiers = [
+classifiers = [
     "Private :: Do Not Upload",
 ]
 description = "Simple repo-review example for testing"

repo_review-0.10.6/.github/matchers/pylint.json

@@ -1,32 +0,0 @@
-{
-  "problemMatcher": [
-    {
-      "severity": "warning",
-      "pattern": [
-        {
-          "regexp": "^([^:]+):(\\d+):(\\d+): ([A-DF-Z]\\d+): \\033\\[[\\d;]+m([^\\033]+).*$",
-          "file": 1,
-          "line": 2,
-          "column": 3,
-          "code": 4,
-          "message": 5
-        }
-      ],
-      "owner": "pylint-warning"
-    },
-    {
-      "severity": "error",
-      "pattern": [
-        {
-          "regexp": "^([^:]+):(\\d+):(\\d+): (E\\d+): \\033\\[[\\d;]+m([^\\033]+).*$",
-          "file": 1,
-          "line": 2,
-          "column": 3,
-          "code": 4,
-          "message": 5
-        }
-      ],
-      "owner": "pylint-error"
-    }
-  ]
-}

repo_review-0.10.6/.readthedocs.yaml

@@ -1,22 +0,0 @@
-# .readthedocs.yaml
-# Read the Docs configuration file
-# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
-
-# Required
-version: 2
-
-build:
-  os: "ubuntu-22.04"
-  tools:
-    python: "3.11"
-
-# Build documentation in the docs/ directory with Sphinx
-sphinx:
-  configuration: docs/conf.py
-
-python:
-  install:
-    - method: pip
-      path: .
-      extra_requirements:
-        - docs

repo_review-0.10.6/noxfile.py

@@ -1,138 +0,0 @@
-from __future__ import annotations
-
-import argparse
-import shutil
-from pathlib import Path
-
-import nox
-
-DIR = Path(__file__).parent.resolve()
-
-nox.needs_version = ">=2024.3.2"
-nox.options.default_venv_backend = "uv|virtualenv"
-nox.options.sessions = ["lint", "pylint", "tests"]
-
-
-@nox.session(reuse_venv=True)
-def run(session: nox.Session) -> None:
-    """
-    Run the program with a few example checks.
-    """
-
-    session.install("-e", ".[cli]")
-    session.install("-e", "tests/test_utilities")
-    session.run("python", "-m", "repo_review", *session.posargs)
-
-
-@nox.session
-def lint(session: nox.Session) -> None:
-    """
-    Run the linter.
-    """
-    session.install("pre-commit")
-    session.run("pre-commit", "run", "--all-files", *session.posargs)
-
-
-@nox.session
-def pylint(session: nox.Session) -> None:
-    """
-    Run PyLint.
-    """
-    # This needs to be installed into the package environment, and is slower
-    # than a pre-commit check
-    session.install("-e.[cli]", "pylint")
-    session.run("pylint", "src", *session.posargs)
-
-
-@nox.session
-def tests(session: nox.Session) -> None:
-    """
-    Run the unit and regular tests.
-    """
-    session.install("-e.[test,cli]")
-    session.run("pytest", *session.posargs)
-
-
-@nox.session(reuse_venv=True)
-def build(session: nox.Session) -> None:
-    """
-    Build an SDist and wheel.
-    """
-
-    build_p = DIR.joinpath("build")
-    if build_p.exists():
-        shutil.rmtree(build_p)
-
-    session.install("build")
-    session.run("python", "-m", "build")
-
-
-@nox.session(venv_backend="none")
-def serve(session: nox.Session) -> None:
-    """
-    Serve the webapp.
-    """
-
-    session.cd("docs")
-    session.log("Serving on http://localhost:8080")
-    session.run("python3", "-m", "http.server", "8080")
-
-
-@nox.session(reuse_venv=True)
-def docs(session: nox.Session) -> None:
-    """
-    Build the docs. Pass "--serve" to serve.
-    """
-
-    parser = argparse.ArgumentParser()
-    parser.add_argument("--serve", action="store_true", help="Serve after building")
-    parser.add_argument(
-        "-b", dest="builder", default="html", help="Build target (default: html)"
-    )
-    args, posargs = parser.parse_known_args(session.posargs)
-
-    if args.builder != "html" and args.serve:
-        session.error("Must not specify non-HTML builder with --serve")
-
-    extra_installs = ["sphinx-autobuild"] if args.serve else []
-
-    session.install("-e.[docs]", *extra_installs)
-    session.chdir("docs")
-
-    if args.builder == "linkcheck":
-        session.run(
-            "sphinx-build", "-b", "linkcheck", ".", "_build/linkcheck", *posargs
-        )
-        return
-
-    shared_args = (
-        "-n",  # nitpicky mode
-        "-T",  # full tracebacks
-        f"-b={args.builder}",
-        ".",
-        f"_build/{args.builder}",
-        *posargs,
-    )
-
-    if args.serve:
-        session.run("sphinx-autobuild", *shared_args)
-    else:
-        session.run("sphinx-build", "--keep-going", *shared_args)
-
-
-@nox.session
-def build_api_docs(session: nox.Session) -> None:
-    """
-    Build (regenerate) API docs.
-    """
-    session.install("sphinx")
-    session.chdir("docs")
-    session.run(
-        "sphinx-apidoc",
-        "-o",
-        "api/",
-        "--module-first",
-        "--no-toc",
-        "--force",
-        "../src/repo_review",
-    )