cpp-linter 1.14.0__tar.gz → 1.14.1__tar.gz

{cpp_linter-1.14.0/cpp_linter.egg-info → cpp_linter-1.14.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cpp-linter
-Version: 1.14.0
+Version: 1.14.1
 Summary: Run clang-format and clang-tidy on a batch of files.
 Author-email: Brendan Doherty <2bndy5@gmail.com>, Xianpeng Shen <xianpeng.shen@gmail.com>
 License-Expression: MIT

{cpp_linter-1.14.0 → cpp_linter-1.14.1}/cpp_linter/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.14.0'
-__version_tuple__ = version_tuple = (1, 14, 0)
+__version__ = version = '1.14.1'
+__version_tuple__ = version_tuple = (1, 14, 1)
 
-__commit_id__ = commit_id = 'g2e850a5d0'
+__commit_id__ = commit_id = 'g44999fc08'

{cpp_linter-1.14.0 → cpp_linter-1.14.1}/cpp_linter/rest_api/github_api.py

@@ -150,6 +150,8 @@ class GithubApiClient(RestApiClient):
                         f"Missing 'filename' key in file:\n{json.dumps(file, indent=2)}"
                     )
                     raise exc
+                if file.get("status", "") == "removed":
+                    continue
                 if not file_filter.is_source_or_ignored(file_name):
                     continue
                 if lines_changed_only > 0 and cast(int, file.get("changes", 0)) == 0:
@@ -160,15 +162,11 @@ class GithubApiClient(RestApiClient):
                 if "patch" not in file:
                     if lines_changed_only > 0:
                         # diff info is needed for further operations
-                        raise KeyError(  # pragma: no cover
+                        raise KeyError(
                             f"{file_name} has no patch info:\n{json.dumps(file, indent=2)}"
                         )
-                    elif (
-                        cast(int, file.get("changes", 0)) == 0
-                    ):  # in case files-changed-only is true
-                        # file was likely renamed without source changes
-                        files.append(FileObj(file_name))  # scan entire file instead
-                        continue
+                    files.append(FileObj(file_name))  # scan entire file instead
+                    continue
                 file_diff = (
                     f"diff --git a/{old_name} b/{file_name}\n"
                     + f"--- a/{old_name}\n+++ b/{file_name}\n"

{cpp_linter-1.14.0 → cpp_linter-1.14.1/cpp_linter.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cpp-linter
-Version: 1.14.0
+Version: 1.14.1
 Summary: Run clang-format and clang-tidy on a batch of files.
 Author-email: Brendan Doherty <2bndy5@gmail.com>, Xianpeng Shen <xianpeng.shen@gmail.com>
 License-Expression: MIT

cpp_linter-1.14.1/cpp_linter.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "1.14.1",
+  "distance": 0,
+  "node": "g44999fc08df9da2d6b268f494b592452fdf74bb1",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-09-22"
+}

{cpp_linter-1.14.0 → cpp_linter-1.14.1}/tests/list_changes/test_get_file_changes.py

@@ -154,3 +154,60 @@ def test_get_changed_files(
             if lines_changed_only == 0:
                 expected.append("include/test/tree.hpp")
             assert file.name in expected
+
+
+@pytest.mark.no_clang
+@pytest.mark.parametrize("event_name", ["pull_request", "push"])
+@pytest.mark.parametrize("status", ["added", "modified", "renamed", "removed"])
+@pytest.mark.parametrize("lines_changed_only", [0, 1, 2])
+def test_paginated_files_without_patches(
+    monkeypatch: pytest.MonkeyPatch,
+    event_name: str,
+    status: str,
+    lines_changed_only: int,
+):
+    """Missing patches permit whole-file linting, but not line filtering."""
+    monkeypatch.setenv("CI", "true")
+    monkeypatch.setenv("GITHUB_EVENT_PATH", "")
+    client = GithubApiClient()
+    client.event_name = event_name
+    client.api_url = TEST_API_URL
+    client.repo = TEST_REPO
+    client.sha = TEST_SHA
+    client.pull_request = TEST_PR
+    endpoint = f"{TEST_API_URL}/repos/{TEST_REPO}/commits/{TEST_SHA}"
+    if event_name == "pull_request":
+        endpoint = f"{TEST_API_URL}/repos/{TEST_REPO}/pulls/{TEST_PR}"
+
+    entries = [
+        {
+            "filename": "src/deleted.cpp",
+            "status": "removed",
+            "changes": 1,
+            "patch": "@@ -1 +0,0 @@\n-int removed;",
+        },
+        {"filename": "src/large.cpp", "status": status, "changes": 1000},
+    ]
+    with requests_mock.Mocker() as mock:
+        mock.get(
+            endpoint,
+            request_headers={"Accept": "application/vnd.github.diff"},
+            status_code=406,
+        )
+        mock.get(
+            endpoint + ("/files" if event_name == "pull_request" else ""),
+            request_headers={"Accept": "application/vnd.github.raw+json"},
+            json=entries if event_name == "pull_request" else {"files": entries},
+        )
+        file_filter = FileFilter(extensions=["cpp"])
+        if lines_changed_only > 0 and status != "removed":
+            with pytest.raises(KeyError, match=r"src/large\.cpp has no patch info"):
+                client.get_list_of_changed_files(file_filter, lines_changed_only)
+        else:
+            files = client.get_list_of_changed_files(file_filter, lines_changed_only)
+            assert [file.name for file in files] == (
+                [] if status == "removed" else ["src/large.cpp"]
+            )
+            for file in files:
+                assert file.lines_added == []
+                assert file.diff_chunks == []

cpp_linter-1.14.0/cpp_linter.egg-info/scm_version.json

@@ -1,8 +0,0 @@
-{
-  "tag": "1.14.0",
-  "distance": 0,
-  "node": "g2e850a5d055b05b3db4bd07960689a3d5286b169",
-  "dirty": false,
-  "branch": "HEAD",
-  "node_date": "2026-09-13"
-}