github-rest-api 0.53.0__tar.gz → 0.53.1__tar.gz

{github_rest_api-0.53.0 → github_rest_api-0.53.1}/.github/workflows/merge-pr.yaml

@@ -42,13 +42,6 @@ on:
 permissions:
   contents: write
   pull-requests: write
-concurrency:
-  # Every run rescans all open PRs, so overlapping runs are pure duplicate work
-  # and the newest run always sees the freshest state. Do not cancel in
-  # progress: a run cancelled midway through the merge API call is worse than a
-  # redundant one.
-  group: merge-pull-request
-  cancel-in-progress: false
 jobs:
   merge_pull_request:
     # A triggering workflow that failed, was cancelled or was skipped left the
@@ -58,6 +51,15 @@ jobs:
     if: |
       github.event_name != 'workflow_run'
       || github.event.workflow_run.conclusion == 'success'
+    concurrency:
+      # Every run rescans all open PRs, so overlapping runs are pure duplicate
+      # work and the newest run always sees the freshest state. Do not cancel
+      # in progress: a run cancelled midway through the merge API call is worse
+      # than a redundant one. This is set on the job rather than the workflow
+      # so that a run whose `if` skips the job never joins the group, where it
+      # would replace (cancel) another PR's pending merge run.
+      group: merge-pull-request
+      cancel-in-progress: false
     uses: legendu-net/.github/.github/workflows/merge-pr.yaml@main
     with:
       dry_run: ${{ inputs.dry_run || false }}

{github_rest_api-0.53.0 → github_rest_api-0.53.1}/PKG-INFO

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

{github_rest_api-0.53.0 → github_rest_api-0.53.1}/github_rest_api/scripts/lint_repo.py

@@ -22,6 +22,7 @@ import argparse
 import os
 import re
 import shutil
+import stat
 import subprocess as sp
 import sys
 from collections.abc import Callable, Sequence
@@ -29,6 +30,7 @@ from dataclasses import dataclass, field
 from pathlib import Path
 from typing import Literal, cast
 
+from dulwich.index import IndexEntry
 from dulwich.repo import Repo
 
 #: The outcome of running one check.
@@ -44,6 +46,12 @@ LANGUAGES = ["python", "rust", "golang", "bash", "fish", "lua", "markdown"]
 def list_tracked_files(root: Path) -> list[str]:
     """List all git-tracked files in a repository.
 
+    Symbolic links are excluded: a link's content is linted through its
+    target (when tracked), and linting the link too would run every check
+    twice on the same content -- or, worse, with conflicting options (e.g. a
+    `CLAUDE.md` -> `AGENTS.md` link would get a different mdformat flavor than
+    its target).
+
     :param root: The root directory of the git repository (not an arbitrary
         subdirectory of one -- the returned paths, and every command a check
         later runs, are relative to this exact directory).
@@ -54,7 +62,10 @@ def list_tracked_files(root: Path) -> list[str]:
         # `surrogateescape` so a non-UTF-8 filename (which git tracks fine)
         # doesn't crash the whole run; it just won't print legibly.
         return sorted(
-            name.decode(errors="surrogateescape") for name in repo.open_index()
+            name.decode(errors="surrogateescape")
+            for name, entry in repo.open_index().items()
+            # A conflicted entry (mid-merge) has no single mode; keep it.
+            if not (isinstance(entry, IndexEntry) and stat.S_ISLNK(entry.mode))
         )
     finally:
         repo.close()

{github_rest_api-0.53.0 → github_rest_api-0.53.1}/pyproject.toml

@@ -4,7 +4,7 @@ requires = [ "hatchling" ]
 
 [project]
 name = "github-rest-api"
-version = "0.53.0"
+version = "0.53.1"
 description = "Simple wrapper of GitHub REST APIs."
 readme = "README.md"
 authors = [ { name = "Ben Du", email = "longendu@yahoo.com" } ]

{github_rest_api-0.53.0 → github_rest_api-0.53.1}/tests/test_lint_repo.py

@@ -58,6 +58,16 @@ def test_list_tracked_files_ignores_untracked_files(tmp_path):
     assert list_tracked_files(tmp_path) == ["tracked.py"]
 
 
+def test_list_tracked_files_excludes_symlinks(tmp_path):
+    _init_repo(tmp_path, {"AGENTS.md": "# Agents\n"})
+    (tmp_path / "CLAUDE.md").symlink_to("AGENTS.md")
+    porcelain.add(str(tmp_path), paths=["CLAUDE.md"])
+    # The link is tracked by git, but only its target should be linted.
+    with Repo(str(tmp_path)) as repo:
+        assert b"CLAUDE.md" in repo.open_index()
+    assert list_tracked_files(tmp_path) == ["AGENTS.md"]
+
+
 def test_list_tracked_files_survives_a_non_utf8_filename(tmp_path):
     # Git tracks arbitrary byte sequences as filenames; decoding with
     # `surrogateescape` must not raise on one that isn't valid UTF-8. Written

{github_rest_api-0.53.0 → github_rest_api-0.53.1}/uv.lock

@@ -542,7 +542,7 @@ wheels = [
 
 [[package]]
 name = "github-rest-api"
-version = "0.53.0"
+version = "0.53.1"
 source = { editable = "." }
 dependencies = [
     { name = "dulwich" },