repo-review 0.12.1__tar.gz → 0.12.2__tar.gz

{repo_review-0.12.1 → repo_review-0.12.2}/.github/workflows/ci.yml

@@ -48,17 +48,20 @@ jobs:
           fetch-depth: 0
           persist-credentials: false
 
+      # Last one is activated
+      # yaml circular import issue on 3.14t on ubuntu
       - uses: actions/setup-python@v5
         with:
           python-version: |
             3.10
             3.11
             3.12
+            3.14
             3.13
           allow-prereleases: true
 
       - name: Setup uv
-        uses: yezz123/setup-uv@v4
+        uses: astral-sh/setup-uv@v6
 
       - name: Install hatch
         run: uv pip install --system hatch
@@ -88,7 +91,7 @@ jobs:
           persist-credentials: false
 
       - name: Setup uv
-        uses: yezz123/setup-uv@v4
+        uses: astral-sh/setup-uv@v6
 
       - uses: actions/setup-python@v5
         with:
@@ -120,4 +123,4 @@ jobs:
       - name: Run repo-review action
         uses: ./
         with:
-          plugins: sp-repo-review==2025.01.22
+          plugins: sp-repo-review==2025.05.02

{repo_review-0.12.1 → repo_review-0.12.2}/.pre-commit-config.yaml

@@ -11,14 +11,14 @@ repos:
         additional_dependencies: [black==24.*]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.9.4"
+    rev: "v0.11.8"
     hooks:
       - id: ruff
         args: ["--fix", "--show-fixes"]
       - id: ruff-format
 
   - repo: https://github.com/rbubley/mirrors-prettier
-    rev: "v3.4.2"
+    rev: "v3.5.3"
     hooks:
       - id: prettier
         types_or: [yaml, markdown, html, css, scss, javascript, json]
@@ -45,7 +45,7 @@ repos:
       - id: rst-inline-touching-normal
 
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.14.1
+    rev: v1.15.0
     hooks:
       - id: mypy
         files: (src|web|tests)
@@ -79,12 +79,12 @@ repos:
         exclude: .pre-commit-config.yaml
 
   - repo: https://github.com/henryiii/validate-pyproject-schema-store
-    rev: 2025.02.03
+    rev: 2025.04.28
     hooks:
       - id: validate-pyproject
 
   - repo: https://github.com/python-jsonschema/check-jsonschema
-    rev: 0.31.1
+    rev: 0.33.0
     hooks:
       - id: check-dependabot
       - id: check-github-workflows

{repo_review-0.12.1 → repo_review-0.12.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: repo_review
-Version: 0.12.1
+Version: 0.12.2
 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
@@ -22,6 +22,7 @@ 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: Programming Language :: Python :: 3.14
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Topic :: Software Development :: Quality Assurance
 Classifier: Typing :: Typed
@@ -128,7 +129,7 @@ repos have some [pre-commit][] check.
 ## Development of repo-review and plugins
 
 This project is intended to be fun and easy to develop and design checks for -
-it requires and uses Python 3.10, and uses a lot of the new features in 3.9 and
+it requires and uses Python 3.10+, and uses a lot of the new features in 3.9 and
 3.10. It's maybe not entirely conventional, but it enables very simple plugin
 development. It works locally, remotely, and in WebAssembly (using
 [Pyodide][]). [See the docs][writing-a-plugin].

{repo_review-0.12.1 → repo_review-0.12.2}/README.md

@@ -90,7 +90,7 @@ repos have some [pre-commit][] check.
 ## Development of repo-review and plugins
 
 This project is intended to be fun and easy to develop and design checks for -
-it requires and uses Python 3.10, and uses a lot of the new features in 3.9 and
+it requires and uses Python 3.10+, and uses a lot of the new features in 3.9 and
 3.10. It's maybe not entirely conventional, but it enables very simple plugin
 development. It works locally, remotely, and in WebAssembly (using
 [Pyodide][]). [See the docs][writing-a-plugin].

{repo_review-0.12.1 → repo_review-0.12.2}/action.yml

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

{repo_review-0.12.1 → repo_review-0.12.2}/docs/index.html

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

{repo_review-0.12.1 → repo_review-0.12.2}/docs/webapp.js

@@ -1,5 +1,5 @@
 const DEFAULT_MSG =
-  "Enter a GitHub repo and branch to review. Runs Python entirely in your browser using WebAssembly. Built with React, MaterialUI, and Pyodide.";
+  "Enter a GitHub repo and branch/tag to review. Runs Python entirely in your browser using WebAssembly. Built with React, MaterialUI, and Pyodide.";
 
 const urlParams = new URLSearchParams(window.location.search);
 const baseurl = window.location.pathname;
@@ -154,6 +154,39 @@ function Results(props) {
   );
 }
 
+async function fetchRepoRefs(repo) {
+  if (!repo) return { branches: [], tags: [] };
+  try {
+    // Fetch both branches and tags from GitHub API
+    const [branchesResponse, tagsResponse] = await Promise.all([
+      fetch(`https://api.github.com/repos/${repo}/branches`),
+      fetch(`https://api.github.com/repos/${repo}/tags`),
+    ]);
+
+    if (!branchesResponse.ok || !tagsResponse.ok) {
+      console.error("Error fetching repo data");
+      return { branches: [], tags: [] };
+    }
+
+    const branches = await branchesResponse.json();
+    const tags = await tagsResponse.json();
+
+    return {
+      branches: branches.map((branch) => ({
+        name: branch.name,
+        type: "branch",
+      })),
+      tags: tags.map((tag) => ({
+        name: tag.name,
+        type: "tag",
+      })),
+    };
+  } catch (error) {
+    console.error("Error fetching repo references:", error);
+    return { branches: [], tags: [] };
+  }
+}
+
 async function prepare_pyodide(deps) {
   const deps_str = deps.map((i) => `"${i}"`).join(", ");
   const pyodide = await loadPyodide();
@@ -196,28 +229,58 @@ class App extends React.Component {
     this.state = {
       results: [],
       repo: urlParams.get("repo") || "",
-      branch: urlParams.get("branch") || "",
+      ref: urlParams.get("ref") || "",
+      refType: urlParams.get("refType") || "branch",
+      refs: { branches: [], tags: [] },
       msg: `<p>${DEFAULT_MSG}</p><h4>Packages:</h4> ${deps_str}`,
       progress: false,
+      loadingRefs: false,
       err_msg: "",
       skip_reason: "",
       url: "",
     };
     this.pyodide_promise = prepare_pyodide(props.deps);
+    this.refInputDebounce = null;
+  }
+
+  async fetchRepoReferences(repo) {
+    if (!repo) return;
+
+    this.setState({ loadingRefs: true });
+    const refs = await fetchRepoRefs(repo);
+    this.setState({
+      refs: refs,
+      loadingRefs: false,
+    });
+  }
+
+  handleRepoChange(repo) {
+    this.setState({ repo });
+
+    // debounce the API call to avoid too many requests
+    clearTimeout(this.refInputDebounce);
+    this.refInputDebounce = setTimeout(() => {
+      this.fetchRepoReferences(repo);
+    }, 500);
+  }
+
+  handleRefChange(ref, refType) {
+    this.setState({ ref, refType });
   }
 
   handleCompute() {
-    if (!this.state.repo || !this.state.branch) {
+    if (!this.state.repo || !this.state.ref) {
       this.setState({ results: [], msg: DEFAULT_MSG });
       window.history.replaceState(null, "", baseurl);
       alert(
-        `Please enter a repo (${this.state.repo}) and branch (${this.state.branch})`,
+        `Please enter a repo (${this.state.repo}) and branch/tag (${this.state.ref})`,
       );
       return;
     }
     const local_params = new URLSearchParams({
       repo: this.state.repo,
-      branch: this.state.branch,
+      ref: this.state.ref,
+      refType: this.state.refType,
     });
     window.history.replaceState(null, "", `${baseurl}?${local_params}`);
     this.setState({
@@ -234,13 +297,13 @@ class App extends React.Component {
           from repo_review.ghpath import GHPath
           from dataclasses import replace
 
-          package = GHPath(repo="${state.repo}", branch="${state.branch}")
+          package = GHPath(repo="${state.repo}", branch="${state.ref}")
           families, checks = process(package)
 
           for v in families.values():
               if v.get("description"):
                   v["description"] = md_as_html(v["description"])
-          checks = [replace(v, err_msg=md_as_html(v.err_msg), skip_reason=md_as_html(v.skip_reason)) for v in checks]
+          checks = [res.md_as_html() for res in checks]
 
           (families, checks)
           `);
@@ -249,7 +312,7 @@ class App extends React.Component {
           this.setState({
             msg: DEFAULT_MSG,
             progress: false,
-            err_msg: "Invalid repository or branch. Please try again.",
+            err_msg: "Invalid repository or branch/tag. Please try again.",
           });
           return;
         }
@@ -288,7 +351,7 @@ class App extends React.Component {
       this.setState({
         results: results,
         families: families,
-        msg: `Results for ${state.repo}@${state.branch}`,
+        msg: `Results for ${state.repo}@${state.ref} (${state.refType})`,
         progress: false,
         err_msg: "",
         url: "",
@@ -300,13 +363,78 @@ class App extends React.Component {
   }
 
   componentDidMount() {
-    if (urlParams.get("repo") && urlParams.get("branch")) {
-      this.handleCompute();
+    if (urlParams.get("repo")) {
+      this.fetchRepoReferences(urlParams.get("repo"));
+
+      if (urlParams.get("ref")) {
+        this.handleCompute();
+      }
     }
   }
 
   render() {
-    const common_branches = ["main", "master", "develop", "stable"];
+    const priorityBranches = ["HEAD", "main", "master", "develop", "stable"];
+    const branchMap = new Map(
+      this.state.refs.branches.map((branch) => [branch.name, branch]),
+    );
+
+    let availableOptions = [];
+
+    // If no repo is entered or API hasn't returned any branches/tags yet,
+    // show all five priority branches.
+    if (
+      this.state.repo === "" ||
+      (this.state.refs.branches.length === 0 &&
+        this.state.refs.tags.length === 0)
+    ) {
+      availableOptions = [
+        { label: "HEAD (default branch)", value: "HEAD", type: "branch" },
+        { label: "main (branch)", value: "main", type: "branch" },
+        { label: "master (branch)", value: "master", type: "branch" },
+        { label: "develop (branch)", value: "develop", type: "branch" },
+        { label: "stable (branch)", value: "stable", type: "branch" },
+      ];
+    } else {
+      const prioritizedBranches = [
+        { label: "HEAD (default branch)", value: "HEAD", type: "branch" },
+      ];
+
+      priorityBranches.slice(1).forEach((branchName) => {
+        if (branchMap.has(branchName)) {
+          prioritizedBranches.push({
+            label: `${branchName} (branch)`,
+            value: branchName,
+            type: "branch",
+          });
+          // Remove from map so it doesn't get added twice.
+          branchMap.delete(branchName);
+        }
+      });
+
+      const otherBranches = [];
+      branchMap.forEach((branch) => {
+        otherBranches.push({
+          label: `${branch.name} (branch)`,
+          value: branch.name,
+          type: "branch",
+        });
+      });
+      otherBranches.sort((a, b) => a.value.localeCompare(b.value));
+
+      const tagOptions = this.state.refs.tags.map((tag) => ({
+        label: `${tag.name} (tag)`,
+        value: tag.name,
+        type: "tag",
+      }));
+      tagOptions.sort((a, b) => a.value.localeCompare(b.value));
+
+      availableOptions = [
+        ...prioritizedBranches,
+        ...otherBranches,
+        ...tagOptions,
+      ];
+    }
+
     return (
       <MyThemeProvider>
         <MaterialUI.CssBaseline />
@@ -326,29 +454,64 @@ class App extends React.Component {
               autoFocus={true}
               onKeyDown={(e) => {
                 if (e.keyCode === 13)
-                  document.getElementById("branch-select").focus();
+                  document.getElementById("ref-select").focus();
               }}
-              onInput={(e) => this.setState({ repo: e.target.value })}
+              onInput={(e) => this.handleRepoChange(e.target.value)}
               defaultValue={urlParams.get("repo")}
               sx={{ flexGrow: 3 }}
             />
             <MaterialUI.Autocomplete
               disablePortal
-              id="branch-select"
-              options={common_branches}
+              id="ref-select"
+              options={availableOptions}
+              loading={this.state.loadingRefs}
               freeSolo={true}
               onKeyDown={(e) => {
                 if (e.keyCode === 13) this.handleCompute();
               }}
-              onInputChange={(e, value) => this.setState({ branch: value })}
-              defaultValue={urlParams.get("branch")}
+              getOptionLabel={(option) =>
+                typeof option === "string" ? option : option.label
+              }
+              renderOption={(props, option) => (
+                <li {...props}>{option.label}</li>
+              )}
+              onInputChange={(e, value) => {
+                // If the user enters free text, treat it as a branch
+                if (typeof value === "string") {
+                  this.handleRefChange(value, "branch");
+                }
+              }}
+              onChange={(e, option) => {
+                if (option) {
+                  if (typeof option === "object") {
+                    this.handleRefChange(option.value, option.type);
+                  } else {
+                    this.handleRefChange(option, "branch");
+                  }
+                }
+              }}
+              defaultValue={urlParams.get("ref")}
               renderInput={(params) => (
                 <MaterialUI.TextField
                   {...params}
-                  label="Branch"
+                  label="Branch/Tag"
                   variant="outlined"
-                  helperText="e.g. main"
-                  sx={{ flexGrow: 2, minWidth: 130 }}
+                  helperText="e.g. HEAD, main, or v1.0.0"
+                  sx={{ flexGrow: 2, minWidth: 200 }}
+                  InputProps={{
+                    ...params.InputProps,
+                    endAdornment: (
+                      <React.Fragment>
+                        {this.state.loadingRefs ? (
+                          <MaterialUI.CircularProgress
+                            color="inherit"
+                            size={20}
+                          />
+                        ) : null}
+                        {params.InputProps.endAdornment}
+                      </React.Fragment>
+                    ),
+                  }}
                 />
               )}
             />
@@ -358,7 +521,7 @@ class App extends React.Component {
               variant="contained"
               size="large"
               disabled={
-                this.state.progress || !this.state.repo || !this.state.branch
+                this.state.progress || !this.state.repo || !this.state.ref
               }
             >
               <MaterialUI.Icon>start</MaterialUI.Icon>

{repo_review-0.12.1 → repo_review-0.12.2}/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.25.1/full/pyodide.js"
+  src="https://cdn.jsdelivr.net/pyodide/v0.27.6/full/pyodide.js"
   crossorigin
 ></script>
 ```

{repo_review-0.12.1 → repo_review-0.12.2}/pyproject.toml

@@ -24,6 +24,7 @@ classifiers = [
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
   "Programming Language :: Python :: 3.13",
+  "Programming Language :: Python :: 3.14",
   "Programming Language :: Python",
   "Topic :: Software Development :: Quality Assurance",
   "Topic :: Software Development :: Libraries :: Python Modules",
@@ -148,7 +149,7 @@ 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"]
+python = ["3.14", "3.13", "3.12", "3.11", "3.10"]
 
 
 [tool.pytest.ini_options]
@@ -201,6 +202,7 @@ messages_control.disable = [
   "used-before-assignment",  # False positive on conditional import
   "unnecessary-ellipsis",  # Not correct for typing
   "import-outside-toplevel",  # better handled elsewhere
+  "duplicate-code", # Triggers incorrectly
 ]
 
 

{repo_review-0.12.1 → repo_review-0.12.2}/src/repo_review/__main__.py

@@ -6,6 +6,7 @@ import json
 import os
 import sys
 import typing
+import urllib.error
 from collections.abc import Mapping, Sequence
 from pathlib import Path
 from typing import Any, Literal
@@ -17,6 +18,7 @@ if typing.TYPE_CHECKING:
 else:
     import rich_click as click
 
+import rich
 import rich.console
 import rich.markdown
 import rich.syntax
@@ -265,7 +267,12 @@ def _remote_path_processor(package: Path) -> Path | GHPath:
         msg = "online repo must be of the form 'gh:org/repo@branch[:path]' (:branch missing)"
         raise click.BadParameter(msg)
     org_repo, branch = org_repo_branch.split("@", maxsplit=1)
-    return GHPath(repo=org_repo, branch=branch, path=p[0] if p else "")
+    try:
+        return GHPath(repo=org_repo, branch=branch, path=p[0] if p else "")
+    except urllib.error.HTTPError as e:
+        rich.print(f"[red][bold]Error[/bold] accessing {e.url}", file=sys.stderr)
+        rich.print(f"[red]{e}", file=sys.stderr)
+        raise SystemExit(1) from None
 
 
 @click.command(context_settings={"help_option_names": ["-h", "--help"]})

{repo_review-0.12.1 → repo_review-0.12.2}/src/repo_review/_version.py

@@ -1,8 +1,13 @@
-# file generated by setuptools_scm
+# file generated by setuptools-scm
 # don't change, don't track in version control
+
+__all__ = ["__version__", "__version_tuple__", "version", "version_tuple"]
+
 TYPE_CHECKING = False
 if TYPE_CHECKING:
-    from typing import Tuple, Union
+    from typing import Tuple
+    from typing import Union
+
     VERSION_TUPLE = Tuple[Union[int, str], ...]
 else:
     VERSION_TUPLE = object
@@ -12,5 +17,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '0.12.1'
-__version_tuple__ = version_tuple = (0, 12, 1)
+__version__ = version = '0.12.2'
+__version_tuple__ = version_tuple = (0, 12, 2)

{repo_review-0.12.1 → repo_review-0.12.2}/tests/test_package.py

@@ -25,7 +25,8 @@ def test_local():
     assert "BSD License" in results.families["general"]["description"]
     assert "[tool.repo-review]" in results.families["validate-pyproject"]["description"]
     for result in results.results:
-        assert result.result
+        if result.result is not None:
+            assert result.result
 
 
 def test_broken_validate_pyproject(tmp_path: Path) -> None:

{repo_review-0.12.1 → repo_review-0.12.2}/docs/api/repo_review.resources.rst

@@ -3,5 +3,5 @@ repo\_review.resources package
 
 .. automodule:: repo_review.resources
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:

{repo_review-0.12.1 → repo_review-0.12.2}/docs/api/repo_review.rst

@@ -3,8 +3,8 @@ repo\_review package
 
 .. automodule:: repo_review
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 Subpackages
 -----------
@@ -22,61 +22,61 @@ repo\_review.checks module
 
 .. automodule:: repo_review.checks
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 repo\_review.families module
 ----------------------------
 
 .. automodule:: repo_review.families
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 repo\_review.fixtures module
 ----------------------------
 
 .. automodule:: repo_review.fixtures
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 repo\_review.ghpath module
 --------------------------
 
 .. automodule:: repo_review.ghpath
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 repo\_review.html module
 ------------------------
 
 .. automodule:: repo_review.html
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 repo\_review.processor module
 -----------------------------
 
 .. automodule:: repo_review.processor
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 repo\_review.schema module
 --------------------------
 
 .. automodule:: repo_review.schema
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members:
 
 repo\_review.testing module
 ---------------------------
 
 .. automodule:: repo_review.testing
    :members:
-   :undoc-members:
    :show-inheritance:
+   :undoc-members: