github-rest-api 0.38.0__tar.gz → 0.39.0__tar.gz

github_rest_api-0.39.0/.claude/settings.json

@@ -0,0 +1,9 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(jj log *)",
+      "Bash(jj bookmark list *)",
+      "Bash(jj diff *)"
+    ]
+  }
+}

{github_rest_api-0.38.0/github_rest_api/actions/github → github_rest_api-0.39.0/.github}/workflows/create_pr_to_main.yml

@@ -13,8 +13,7 @@ jobs:
           curl -LsSf https://astral.sh/uv/install.sh | sudo env UV_INSTALL_DIR="/usr/local/bin" sh
       - name: Create PR To main
         run: | 
-          curl -sSL https://raw.githubusercontent.com/legendu-net/github_actions_scripts/main/create_pull_request.py \
-            | uv run --script - \
+          uvx --from github-rest-api@latest create_pull_request \
             --head-branch ${{ github.ref_name }} \
             --base-branch main \
             --token ${{ secrets.GITHUBACTIONS }}

{github_rest_api-0.38.0 → github_rest_api-0.39.0}/.github/workflows/remove_branch.yml

@@ -12,8 +12,7 @@ jobs:
           curl -LsSf https://astral.sh/uv/install.sh | sudo env UV_INSTALL_DIR="/usr/local/bin" sh
       - name: Remove Branches
         run: | 
-          curl -sSL https://raw.githubusercontent.com/legendu-net/github_actions_scripts/main/remove_branch.py \
-            | uv run --script - \
+          uvx --from github-rest-api@latest remove_branch \
             --repo ${{ github.repository }} \
             --pattern '^(?!main$)' \
             --token ${{ secrets.GITHUBACTIONS }}

{github_rest_api-0.38.0 → github_rest_api-0.39.0}/GEMINI.md

@@ -15,9 +15,9 @@ A simple Python wrapper for GitHub REST APIs, optimized for use in GitHub Action
   - **psutil**: For system and process utilities.
 - **Architecture:**
   - `github_rest_api/github.py`: Contains the `GitHub` class for handling API requests (GET, POST, DELETE, PUT, PATCH).
-  - `github_rest_api/actions/github/`: Utilities for GitHub actions like creating pull requests, managing releases, and adding repositories.
-  - `github_rest_api/actions/container/`: Utilities for building and configuring container images.
-  - `github_rest_api/actions/cargo/`: Specific support for Rust projects (benchmarking and profiling).
+  - `github_rest_api/scripts/github/`: Utilities for GitHub actions like creating pull requests, managing releases, and adding repositories.
+  - `github_rest_api/scripts/container/`: Utilities for building and configuring container images.
+  - `github_rest_api/scripts/cargo/`: Specific support for Rust projects (benchmarking and profiling).
   - `github_rest_api/utils.py`: General-purpose utilities (versioning, partitioning).
 
 ## Building and Running

{github_rest_api-0.38.0 → github_rest_api-0.39.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: github-rest-api
-Version: 0.38.0
+Version: 0.39.0
 Summary: Simple wrapper of GitHub REST APIs.
 Author-email: Ben Du <longendu@yahoo.com>
 Classifier: Programming Language :: Python :: 3 :: Only

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/container/build_container_images.py

@@ -132,6 +132,14 @@ def _build_image(
         _push_image(f"{image}:{tag}", tool=tool)
 
 
+def _validate_paths_exist(paths: Sequence[str], label: str) -> None:
+    missing = [p for p in paths if not Path(p).exists()]
+    if missing:
+        raise FileNotFoundError(
+            f"\nError: the following {label} do not exist:\n{'\n'.join(missing)}"
+        )
+
+
 def build_images(
     commit1: str,
     commit2: str,
@@ -140,6 +148,8 @@ def build_images(
     tool: str = "podman",
     registry: str = "quay.io/legendu",
 ):
+    _validate_paths_exist(image_dirs, "image dirs")
+    _validate_paths_exist(paths_monitoring, "monitored paths")
     if not has_relevant_changes(commit1, commit2, image_dirs, paths_monitoring):
         print(
             f"Skip building {tool} images as there are no relevant changes between {
@@ -160,7 +170,9 @@ def build_images(
             print(f"Error building {image_dir}: {e}", flush=True)
             failures.append(image_dir)
     if failures:
-        sys.exit(f"\n\nError: failed to build images: {', '.join(failures)}\n")
+        raise RuntimeError(
+            f"\n\nError: failed to build images: {', '.join(failures)}\n"
+        )
 
 
 def parse_args():
@@ -209,11 +221,11 @@ def parse_args():
         help="Explicit list of image directories to build.",
     )
     group.add_argument(
-        "-f",
+        "-y",
         "--yaml-image-dirs",
         dest="yaml_image_dirs",
         default=None,
-        metavar="FILE",
+        metavar="YAML_FILE",
         help="Path to a YAML file containing a list of image dirs to build.",
     )
     parser.add_argument(
@@ -243,17 +255,22 @@ def _resolve_image_dirs(args: argparse.Namespace) -> list[str]:
         return [s for item in data if (s := item.strip())]
 
 
-def main():
+def main() -> int:
     args = parse_args()
-    build_images(
-        args.commit1,
-        args.commit2,
-        _resolve_image_dirs(args),
-        paths_monitoring=args.paths_monitoring,
-        tool=args.tool,
-        registry=args.registry,
-    )
+    try:
+        build_images(
+            args.commit1,
+            args.commit2,
+            _resolve_image_dirs(args),
+            paths_monitoring=args.paths_monitoring,
+            tool=args.tool,
+            registry=args.registry,
+        )
+    except Exception as e:
+        print(str(e), file=sys.stderr)
+        return 1
+    return 0
 
 
 if __name__ == "__main__":
-    main()
+    sys.exit(main())

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/container/config_container.py

@@ -1,6 +1,7 @@
 import argparse
 import json
 import shutil
+import sys
 import tomllib
 import tomli_w
 from pathlib import Path
@@ -69,14 +70,19 @@ def parse_args(args=None):
     return parser.parse_args(args=args)
 
 
-def main():
+def main() -> int:
     args = parse_args()
     runtimes = set(args.runtime) if args.runtime else {"docker", "podman"}
-    if "docker" in runtimes:
-        config_docker(args.docker_data_root)
-    if "podman" in runtimes:
-        config_podman(args.podman_graphroot)
+    try:
+        if "docker" in runtimes:
+            config_docker(args.docker_data_root)
+        if "podman" in runtimes:
+            config_podman(args.podman_graphroot)
+    except Exception as e:
+        print(f"Error: {e}", file=sys.stderr)
+        return 1
+    return 0
 
 
 if __name__ == "__main__":
-    main()
+    sys.exit(main())

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/container/update_version_containerfile.py

@@ -1,6 +1,7 @@
 import argparse
 import datetime
 import os
+import sys
 from pathlib import Path
 import re
 from dulwich import porcelain
@@ -124,22 +125,27 @@ def checkout_branch(repo: str):
     porcelain.checkout(repo=".", target=branch)
 
 
-def main():
+def main() -> int:
     args = parse_args()
-    if has_open_pr(head_prefix=_branch_prefix(args.repo)):
-        return
-    checkout_branch(args.repo)
-    version = parse_latest_version(repo=args.repo)
-    if args.next_minor_or_strip_patch is not None:
-        version = next_minor_or_strip_patch(version, args.next_minor_or_strip_patch)
-    update_version(
-        containerfile=args.containerfile,
-        version=version,
-        pattern=args.pattern,
-        replace=args.replace,
-    )
-    push_changes(repo=args.repo, token=args.token)
+    try:
+        if has_open_pr(head_prefix=_branch_prefix(args.repo)):
+            return 0
+        checkout_branch(args.repo)
+        version = parse_latest_version(repo=args.repo)
+        if args.next_minor_or_strip_patch is not None:
+            version = next_minor_or_strip_patch(version, args.next_minor_or_strip_patch)
+        update_version(
+            containerfile=args.containerfile,
+            version=version,
+            pattern=args.pattern,
+            replace=args.replace,
+        )
+        push_changes(repo=args.repo, token=args.token)
+    except Exception as e:
+        print(str(e), file=sys.stderr)
+        return 1
+    return 0
 
 
 if __name__ == "__main__":
-    main()
+    sys.exit(main())

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/github/add_github_repo.py

@@ -77,7 +77,7 @@ def parse_args(args=None, namespace=None):
 def _validate_repo(repo: str) -> None:
     parts = repo.split("/")
     if len(parts) != 2 or not parts[0] or not parts[1]:
-        sys.exit(f"Invalid repo format '{repo}'. Expected 'owner/repo'.")
+        raise ValueError(f"Invalid repo format '{repo}'. Expected 'owner/repo'.")
 
 
 def _create_remote_repo(
@@ -142,7 +142,7 @@ def add_github_repo(
     if not token:
         token = getpass.getpass("Please enter your GitHub token: ")
         if not token:
-            sys.exit(
+            raise ValueError(
                 "No GitHub token is provided (via $GITHUB_TOKEN, --token or at prompt)."
             )
     repo = repo.strip()
@@ -177,18 +177,23 @@ def _add_workflow(path: Path, language: str, workflow_dir: Path | None = None) -
             shutil.copy2(yaml, dir_dest)
 
 
-def main():
+def main() -> int:
     args = parse_args()
-    add_github_repo(
-        repo=args.repo,
-        private=args.private,
-        language=args.language,
-        is_owner_user=args.is_owner_user,
-        dir_=args.dir,
-        token=args.token,
-        branches=args.branches,
-    )
+    try:
+        add_github_repo(
+            repo=args.repo,
+            private=args.private,
+            language=args.language,
+            is_owner_user=args.is_owner_user,
+            dir_=args.dir,
+            token=args.token,
+            branches=args.branches,
+        )
+    except Exception as e:
+        print(str(e), file=sys.stderr)
+        return 1
+    return 0
 
 
 if __name__ == "__main__":
-    main()
+    sys.exit(main())

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/github/create_pull_request.py

@@ -4,6 +4,7 @@ The branch is updated (using dev) before creating the PR.
 
 from argparse import ArgumentParser, Namespace
 import os
+import sys
 from github_rest_api import Repository
 
 
@@ -36,7 +37,7 @@ def parse_args(args=None, namespace=None) -> Namespace:
     return parser.parse_args(args=args, namespace=namespace)
 
 
-def main():
+def main() -> int:
     """Main entrance of the script,
     which creates a PR from the specified branch to dev.
     The branch is updated (using dev) before creating the PR.
@@ -44,7 +45,7 @@ def main():
     args = parse_args()
     # skip branches with the pattern _*
     if args.head_branch.startswith("_"):
-        return
+        return 0
     repo = Repository(args.token, os.environ["GITHUB_REPOSITORY"])
     repo.create_pull_request(
         {
@@ -53,7 +54,8 @@ def main():
             "title": f"Merge {args.head_branch} Into {args.base_branch}",
         },
     )
+    return 0
 
 
 if __name__ == "__main__":
-    main()
+    sys.exit(main())

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/github/release_on_github.py

@@ -1,93 +1,44 @@
 import os
+import re
 import sys
 import argparse
-import tomllib
 import getpass
 from pathlib import Path
-from dulwich.repo import Repo
-from dulwich.errors import NotGitRepository
 from github_rest_api import Repository
+from github_rest_api.scripts.utils import (
+    find_project_root,
+    get_project_version,
+    get_repo,
+)
 
 
-def find_root(path: Path | None = None) -> Path | None:
-    if path is None:
-        path = Path.cwd()
-    while path != path.parent:
-        if (path / ".git").exists():
-            return path
-        path = path.parent
-    return None
-
-
-def get_version(root: Path) -> str:
-    pyproject = root / "pyproject.toml"
-    if pyproject.exists():
-        with pyproject.open("rb") as f:
-            data = tomllib.load(f)
-            return data.get("project", {}).get("version", "")
-    cargo = root / "Cargo.toml"
-    if cargo.exists():
-        with cargo.open("rb") as f:
-            data = tomllib.load(f)
-            return data.get("package", {}).get("version", "")
-    return ""
-
-
-def get_repo(root: Path) -> str | None:
-    pyproject = root / "pyproject.toml"
-    if pyproject.exists():
-        with pyproject.open("rb") as f:
-            data = tomllib.load(f)
-            repo_url = data.get("project", {}).get("urls", {}).get("Repository")
-            if repo_url:
-                return (
-                    repo_url.split("github.com/")[-1].removesuffix(".git").rstrip("/")
-                )
-    # Fallback to git remote
-    try:
-        repo = Repo(root)
-        config = repo.get_config()
-        output = config.get((b"remote", b"origin"), b"url").decode().strip()
-        if "github.com" in output:
-            if output.startswith("git@"):
-                return output.split("github.com:")[-1].replace(".git", "").rstrip("/")
-            return output.split("github.com/")[-1].replace(".git", "").rstrip("/")
-    except (NotGitRepository, KeyError):
-        pass
-    return None
+def _get_release_tag(tag: str, root: Path, validate: bool = True) -> str:
+    tag = tag.strip()
+    if not tag:
+        tag = get_project_version(root).strip()
+        if not tag:
+            raise ValueError(
+                "Could not find project version to use as tag. Please specify a tag."
+            )
 
+    if not validate:
+        return tag
 
-def parse_args(args=None, namespace=None):
-    parser = argparse.ArgumentParser(
-        description="Make a release of the project on GitHub."
-    )
-    parser.add_argument(
-        "-b",
-        "--branch",
-        default="main",
-        help="The branch (default to main) from which to make the release.",
-    )
-    parser.add_argument(
-        "-t",
-        "--tag",
-        default="",
-        help="The tag for the release. If not specified, the version from project configuration is used.",
-    )
-    parser.add_argument(
-        "-n",
-        "--notes",
-        default="",
-        help="Notes for the release. If not specified, it's auto generated.",
-    )
-    parser.add_argument(
-        "--token",
-        default="",
-        help="GitHub token. If not specified, the GITHUB_TOKEN environment variable is used.",
-    )
-    return parser.parse_args(args=args, namespace=namespace)
+    # matches v?X.Y.Z and optionally pre-releases (e.g., v1.2.3-alpha.1) or build metadata
+    semver_pattern = r"v?\d+\.\d+\.\d+(?:-[a-zA-Z0-9.-]+)?(?:\+[a-zA-Z0-9.-]+)?"
+    if not re.fullmatch(semver_pattern, tag):
+        raise ValueError(
+            f"Tag '{tag}' is not in a semantic versioning format. Use --no-validate to skip."
+        )
+    normalized = tag if tag.startswith("v") else f"v{tag}"
+    if normalized != tag:
+        print(f"Tag normalized from '{tag}' to '{normalized}'.")
+    return normalized
 
 
-def release_on_github(token: str, branch: str, tag: str = "", notes: str = "") -> None:
+def release_on_github(
+    token: str, branch: str, tag: str = "", notes: str = "", validate: bool = True
+) -> None:
     """Make a release of the project on GitHub.
 
     The command should be run under the root or a sub-dir of the project.
@@ -99,28 +50,24 @@ def release_on_github(token: str, branch: str, tag: str = "", notes: str = "") -
         If not specified, it's auto generated.
     :param token: GitHub token.
         If not specified, the GITHUB_TOKEN environment variable is used.
+    :param validate: If True, validate the tag against semantic versioning format before creating the release.
     """
     token = token or os.getenv("GITHUB_TOKEN", "")
     if not token:
         token = getpass.getpass("Please enter your GitHub token: ")
         if not token:
-            sys.exit(
+            raise ValueError(
                 "No GitHub token is provided (via $GITHUB_TOKEN, --token or at prompt)."
             )
-    root = find_root()
+    root = find_project_root()
     if not root:
-        sys.exit("Could not find project root (no .git found).")
+        raise FileNotFoundError("Could not find project root (no .git found).")
 
-    if not tag:
-        tag = get_version(root)
-        if not tag:
-            sys.exit(
-                "Could not find project version to use as tag. Please specify a tag."
-            )
+    tag = _get_release_tag(tag, root, validate=validate)
 
     repo_name = get_repo(root)
     if not repo_name:
-        sys.exit("Could not find GitHub repository name.")
+        raise ValueError("Could not find GitHub repository name.")
 
     repo = Repository(token=token, repo=repo_name)
     data = {
@@ -136,12 +83,57 @@ def release_on_github(token: str, branch: str, tag: str = "", notes: str = "") -
     print(f"Successfully created release {tag} on {repo_name}.")
 
 
-def main():
-    args = parse_args()
-    release_on_github(
-        token=args.token, branch=args.branch, tag=args.tag, notes=args.notes
+def parse_args(args=None, namespace=None):
+    parser = argparse.ArgumentParser(
+        description="Make a release of the project on GitHub."
+    )
+    parser.add_argument(
+        "-b",
+        "--branch",
+        default="main",
+        help="The branch (default to main) from which to make the release.",
     )
+    parser.add_argument(
+        "-t",
+        "--tag",
+        default="",
+        help="The tag for the release. If not specified, the version from project configuration is used.",
+    )
+    parser.add_argument(
+        "-n",
+        "--notes",
+        default="",
+        help="Notes for the release. If not specified, it's auto generated.",
+    )
+    parser.add_argument(
+        "--token",
+        default="",
+        help="GitHub token. If not specified, the GITHUB_TOKEN environment variable is used.",
+    )
+    parser.add_argument(
+        "--no-validate",
+        dest="validate",
+        action="store_false",
+        help="Skip release tag format validation and normalization. Use it exactly as provided.",
+    )
+    return parser.parse_args(args=args, namespace=namespace)
+
+
+def main() -> int:
+    args = parse_args()
+    try:
+        release_on_github(
+            token=args.token,
+            branch=args.branch,
+            tag=args.tag,
+            notes=args.notes,
+            validate=args.validate,
+        )
+    except Exception as e:
+        print(str(e), file=sys.stderr)
+        return 1
+    return 0
 
 
 if __name__ == "__main__":
-    main()
+    sys.exit(main())

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/github/remove_branch.py

@@ -1,5 +1,6 @@
 import argparse
 import re
+import sys
 import datetime
 from github_rest_api import Repository
 
@@ -93,10 +94,15 @@ def remove_branch(token: str, repo: str, pattern: str) -> None:
         repository.delete_branch(branch_name)
 
 
-def main():
+def main() -> int:
     args = parse_args()
-    remove_branch(token=args.token, repo=args.repo, pattern=args.pattern)
+    try:
+        remove_branch(token=args.token, repo=args.repo, pattern=args.pattern)
+    except Exception as e:
+        print(str(e), file=sys.stderr)
+        return 1
+    return 0
 
 
 if __name__ == "__main__":
-    main()
+    sys.exit(main())

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/github/workflows/create_pr_dev_to_main.yml

@@ -12,8 +12,7 @@ jobs:
           curl -LsSf https://astral.sh/uv/install.sh | sudo env UV_INSTALL_DIR="/usr/local/bin" sh
       - name: Create PR From dev To main
         run: | 
-          curl -sSL https://raw.githubusercontent.com/legendu-net/github_actions_scripts/main/create_pull_request.py \
-            | uv run --script - \
+          uvx --from github-rest-api@latest create_pull_request \
             --head-branch dev \
             --base-branch main \
             --token ${{ secrets.GITHUBACTIONS }}

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/github/workflows/create_pr_to_dev.yml

@@ -14,8 +14,7 @@ jobs:
           curl -LsSf https://astral.sh/uv/install.sh | sudo env UV_INSTALL_DIR="/usr/local/bin" sh
       - name: Create PR to dev
         run: | 
-          curl -sSL https://raw.githubusercontent.com/legendu-net/github_actions_scripts/main/create_pull_request.py \
-            | uv run --script - \
+          uvx --from github-rest-api@latest create_pull_request \
             --head-branch ${{ github.ref_name }} \
             --base-branch dev \
             --token ${{ secrets.GITHUBACTIONS }}

{github_rest_api-0.38.0/.github → github_rest_api-0.39.0/github_rest_api/scripts/github}/workflows/create_pr_to_main.yml

@@ -13,8 +13,7 @@ jobs:
           curl -LsSf https://astral.sh/uv/install.sh | sudo env UV_INSTALL_DIR="/usr/local/bin" sh
       - name: Create PR To main
         run: | 
-          curl -sSL https://raw.githubusercontent.com/legendu-net/github_actions_scripts/main/create_pull_request.py \
-            | uv run --script - \
+          uvx --from github-rest-api@latest create_pull_request \
             --head-branch ${{ github.ref_name }} \
             --base-branch main \
             --token ${{ secrets.GITHUBACTIONS }}

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/github/workflows/remove_branch.yml

@@ -12,8 +12,7 @@ jobs:
           curl -LsSf https://astral.sh/uv/install.sh | sudo env UV_INSTALL_DIR="/usr/local/bin" sh
       - name: Remove Branches
         run: | 
-          curl -sSL https://raw.githubusercontent.com/legendu-net/github_actions_scripts/main/remove_branch.py \
-            | uv run --script - \
+          uvx --from github-rest-api@latest remove_branch \
             --repo ${{ github.repository }} \
             --pattern '^(?!(main|dev)$)' \
             --token ${{ secrets.GITHUBACTIONS }}

{github_rest_api-0.38.0/github_rest_api/actions → github_rest_api-0.39.0/github_rest_api/scripts}/utils.py

@@ -1,7 +1,9 @@
 """Util functions for GitHub actions."""
 
-from typing import Iterable
+import tomllib
+from typing import Any, Iterable
 from pathlib import Path
+from collections.abc import Sequence
 import random
 from dulwich import porcelain
 from dulwich.repo import Repo
@@ -73,3 +75,68 @@ def commit_profiling(prof_dir: str | Path):
     """
     porcelain.add(paths=prof_dir)
     porcelain.commit(message="Updating profiling results.")
+
+
+def find_project_root(path: Path | None = None) -> Path | None:
+    if path is None:
+        path = Path.cwd()
+    while path != path.parent:
+        if (path / ".git").exists():
+            return path
+        path = path.parent
+    return None
+
+
+def get_toml_value(path: Path, keys: Sequence[str]) -> Any:
+    if not keys or not path.exists():
+        return None
+    try:
+        with path.open("rb") as f:
+            data = tomllib.load(f)
+    except Exception:
+        return None
+    for key in keys[:-1]:
+        data = data.get(key, {}) if isinstance(data, dict) else {}
+    return data.get(keys[-1]) if isinstance(data, dict) else None
+
+
+def get_project_version(root: Path) -> str:
+    for path, keys in [
+        (root / "pyproject.toml", ["project", "version"]),
+        (root / "Cargo.toml", ["package", "version"]),
+    ]:
+        version = get_toml_value(path, keys)
+        if isinstance(version, str) and version.strip():
+            return version
+    return ""
+
+
+def parse_github_repo(url: str) -> str:
+    delimiters = ["github.com/", "github.com:"]
+    delim = next((d for d in delimiters if d in url), "")
+    if not delim:
+        return ""
+    return url.split(delim)[-1].rstrip("/").removesuffix(".git")
+
+
+def _get_repo_from_toml(path: Path, keys: Sequence[str]) -> str:
+    value = get_toml_value(path, keys)
+    return parse_github_repo(value) if isinstance(value, str) else ""
+
+
+def get_repo(root: Path) -> str | None:
+    if repo := _get_repo_from_toml(
+        root / "pyproject.toml", ["project", "urls", "Repository"]
+    ):
+        return repo
+    if repo := _get_repo_from_toml(root / "Cargo.toml", ["package", "repository"]):
+        return repo
+    try:
+        repo = Repo(root)
+        config = repo.get_config()
+        url = config.get((b"remote", b"origin"), b"url").decode().strip()
+        if repo_name := parse_github_repo(url):
+            return repo_name
+    except Exception:
+        pass
+    return None

{github_rest_api-0.38.0 → github_rest_api-0.39.0}/pyproject.toml

@@ -4,7 +4,7 @@ requires = [ "hatchling" ]
 
 [project]
 name = "github-rest-api"
-version = "0.38.0"
+version = "0.39.0"
 description = "Simple wrapper of GitHub REST APIs."
 readme = "README.md"
 authors = [ { name = "Ben Du", email = "longendu@yahoo.com" } ]
@@ -23,13 +23,13 @@ dependencies = [
   "tenacity>=9.1.4",
   "tomli-w>=1",
 ]
-scripts.add_github_repo = "github_rest_api.actions.github.add_github_repo:main"
-scripts.build_container_images = "github_rest_api.actions.container.build_container_images:main"
-scripts.config_container = "github_rest_api.actions.container.config_container:main"
-scripts.create_pull_request = "github_rest_api.actions.github.create_pull_request:main"
-scripts.release_on_github = "github_rest_api.actions.github.release_on_github:main"
-scripts.remove_branch = "github_rest_api.actions.github.remove_branch:main"
-scripts.update_version_containerfile = "github_rest_api.actions.container.update_version_containerfile:main"
+scripts.add_github_repo = "github_rest_api.scripts.github.add_github_repo:main"
+scripts.build_container_images = "github_rest_api.scripts.container.build_container_images:main"
+scripts.config_container = "github_rest_api.scripts.container.config_container:main"
+scripts.create_pull_request = "github_rest_api.scripts.github.create_pull_request:main"
+scripts.release_on_github = "github_rest_api.scripts.github.release_on_github:main"
+scripts.remove_branch = "github_rest_api.scripts.github.remove_branch:main"
+scripts.update_version_containerfile = "github_rest_api.scripts.container.update_version_containerfile:main"
 
 [dependency-groups]
 dev = [

{github_rest_api-0.38.0 → github_rest_api-0.39.0}/tests/test_build_container_images.py

@@ -1,6 +1,6 @@
 from pathlib import Path
 from unittest.mock import patch
-from github_rest_api.actions.container.build_container_images import (
+from github_rest_api.scripts.container.build_container_images import (
     has_relevant_changes,
 )
 
@@ -19,7 +19,7 @@ def test_has_relevant_changes_empty_commits():
 
 
 @patch(
-    "github_rest_api.actions.container.build_container_images.changed_files_between",
+    "github_rest_api.scripts.container.build_container_images.changed_files_between",
     return_value=[],
 )
 def test_has_relevant_changes_no_changes(mock_changed):
@@ -28,7 +28,7 @@ def test_has_relevant_changes_no_changes(mock_changed):
 
 
 @patch(
-    "github_rest_api.actions.container.build_container_images.changed_files_between",
+    "github_rest_api.scripts.container.build_container_images.changed_files_between",
     return_value=[IRRELEVANT_FILE],
 )
 def test_has_relevant_changes_only_irrelevant_files(mock_changed):
@@ -37,7 +37,7 @@ def test_has_relevant_changes_only_irrelevant_files(mock_changed):
 
 
 @patch(
-    "github_rest_api.actions.container.build_container_images.changed_files_between",
+    "github_rest_api.scripts.container.build_container_images.changed_files_between",
     return_value=[RELEVANT_FILE],
 )
 def test_has_relevant_changes_with_relevant_file(mock_changed):
@@ -46,7 +46,7 @@ def test_has_relevant_changes_with_relevant_file(mock_changed):
 
 
 @patch(
-    "github_rest_api.actions.container.build_container_images.changed_files_between",
+    "github_rest_api.scripts.container.build_container_images.changed_files_between",
     return_value=[IRRELEVANT_FILE, RELEVANT_FILE],
 )
 def test_has_relevant_changes_mixed_files(mock_changed):
@@ -55,7 +55,7 @@ def test_has_relevant_changes_mixed_files(mock_changed):
 
 
 @patch(
-    "github_rest_api.actions.container.build_container_images.changed_files_between",
+    "github_rest_api.scripts.container.build_container_images.changed_files_between",
     return_value=[RELEVANT_FILE],
 )
 def test_has_relevant_changes_bytes_commits(mock_changed):

github_rest_api-0.39.0/tests/test_release_on_github.py

@@ -0,0 +1,62 @@
+from pathlib import Path
+from unittest.mock import patch
+import pytest
+from github_rest_api.scripts.github.release_on_github import _get_release_tag
+
+ROOT = Path(".")
+
+
+def test_tag_already_normalized():
+    assert _get_release_tag("v1.2.3", ROOT) == "v1.2.3"
+
+
+def test_tag_bare_gets_v_prefix(capsys):
+    assert _get_release_tag("1.2.3", ROOT) == "v1.2.3"
+    assert "normalized" in capsys.readouterr().out
+
+
+def test_tag_with_surrounding_whitespace():
+    assert _get_release_tag("  v1.2.3  ", ROOT) == "v1.2.3"
+
+
+def test_empty_tag_falls_back_to_version():
+    with patch(
+        "github_rest_api.scripts.github.release_on_github.get_project_version",
+        return_value="1.2.3",
+    ):
+        assert _get_release_tag("", ROOT) == "v1.2.3"
+
+
+def test_whitespace_tag_falls_back_to_version():
+    with patch(
+        "github_rest_api.scripts.github.release_on_github.get_project_version",
+        return_value="1.2.3",
+    ):
+        assert _get_release_tag("   ", ROOT) == "v1.2.3"
+
+
+def test_empty_tag_no_version_exits():
+    with patch(
+        "github_rest_api.scripts.github.release_on_github.get_project_version",
+        return_value="",
+    ):
+        with pytest.raises(ValueError):
+            _get_release_tag("", ROOT)
+
+
+def test_invalid_tag_format_exits():
+    with pytest.raises(ValueError):
+        _get_release_tag("invalid", ROOT)
+
+
+def test_no_validate_skips_validation_and_normalization():
+    assert _get_release_tag("invalid", ROOT, validate=False) == "invalid"
+    assert _get_release_tag("1.2.3", ROOT, validate=False) == "1.2.3"
+
+
+def test_prerelease_tag():
+    assert _get_release_tag("v1.2.3-alpha.1", ROOT) == "v1.2.3-alpha.1"
+
+
+def test_build_metadata_tag():
+    assert _get_release_tag("1.2.3+build", ROOT) == "v1.2.3+build"

{github_rest_api-0.38.0 → github_rest_api-0.39.0}/uv.lock

@@ -166,7 +166,7 @@ wheels = [
 
 [[package]]
 name = "github-rest-api"
-version = "0.38.0"
+version = "0.38.2"
 source = { editable = "." }
 dependencies = [
     { name = "dulwich" },