cpp-linter 1.11.3__tar.gz → 1.11.4__tar.gz

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/.github/workflows/analyze-workflows.yml

@@ -12,11 +12,11 @@ jobs:
   analyze-ci:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+      - uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
         with:
           persist-credentials: false
       - name: Set up Python
-        uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
+        uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
         with:
           python-version: '3.x'
       - name: Run zizmor

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/.github/workflows/release.yml

@@ -20,19 +20,19 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+      - uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
         # use fetch --all for setuptools_scm to work
         with:
           fetch-depth: 0
           persist-credentials: false
       - name: Set up Python
-        uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
+        uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
         with:
           python-version: '3.x'
       - name: Build distributions
         run: pipx run build -o dist
       - name: Upload builds as artifacts
-        uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
+        uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5.0.0
         with:
           path: dist
           name: wheels
@@ -45,13 +45,13 @@ jobs:
       id-token: write
     steps:
       - name: Download distributions
-        uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
+        uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
         with:
           name: wheels
           path: dist
       - name: Set up Python
         if: ${{ !startsWith(github.ref, 'refs/tags/') }}
-        uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
+        uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
         with:
           python-version: '3.x'
       - name: Check distributions

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/.github/workflows/tests.yml

@@ -36,11 +36,11 @@ jobs:
 
     runs-on: ${{ matrix.os }}
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+      - uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
         with:
           persist-credentials: false
 
-      - uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
+      - uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
         with:
           python-version: 3.x
 
@@ -60,7 +60,7 @@ jobs:
       #     sudo cmake --install build
 
       - name: Install uv
-        uses: astral-sh/setup-uv@d0cc045d04ccac9d8b7881df0226f9e82c39688e # v6.8.0
+        uses: astral-sh/setup-uv@1e862dfacbd1d6d858c55d9b792c756523627244 # v7.1.4
         with:
           enable-cache: true
           cache-dependency-glob: "uv.lock"
@@ -112,7 +112,7 @@ jobs:
         run: uvx nox -s test-all -- -m "${PYTEST_ARGS}"
 
       - name: Upload coverage data
-        uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
+        uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5.0.0
         with:
           name: coverage-data-${{ runner.os }}-${{ matrix.version }}
           path: .coverage*
@@ -122,23 +122,23 @@ jobs:
     needs: [test]
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+      - uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
         with:
           persist-credentials: false
 
       - name: Download all artifacts
-        uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
+        uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
         with:
           pattern: coverage-data-*
           merge-multiple: true
 
       - name: Setup python
-        uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
+        uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
         with:
           python-version: '3.x'
 
       - name: Install uv
-        uses: astral-sh/setup-uv@d0cc045d04ccac9d8b7881df0226f9e82c39688e # v6.8.0
+        uses: astral-sh/setup-uv@1e862dfacbd1d6d858c55d9b792c756523627244 # v7.1.4
         with:
           enable-cache: true
           cache-dependency-glob: "uv.lock"
@@ -147,7 +147,7 @@ jobs:
         run: uvx nox -s coverage
 
       - name: Upload comprehensive coverage HTML report
-        uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
+        uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5.0.0
         with:
           name: coverage-report
           path: htmlcov/

{cpp_linter-1.11.3/cpp_linter.egg-info → cpp_linter-1.11.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cpp-linter
-Version: 1.11.3
+Version: 1.11.4
 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.11.3 → cpp_linter-1.11.4}/cpp_linter/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '1.11.3'
-__version_tuple__ = version_tuple = (1, 11, 3)
+__version__ = version = '1.11.4'
+__version_tuple__ = version_tuple = (1, 11, 4)
 
-__commit_id__ = commit_id = 'gf416f75ac'
+__commit_id__ = commit_id = 'ga846191b3'

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/cpp_linter/clang_tools/clang_format.py

@@ -187,6 +187,7 @@ def run_clang_format(
         file_obj.range_of_changed_lines(lines_changed_only, get_ranges=True),
     )
     for span in ranges:
+        # line range (start and end) is inclusive
         cmds.append(f"--lines={span[0]}:{span[1]}")
     cmds.append(PurePath(file_obj.name).as_posix())
     logger.info('Running "%s"', " ".join(cmds))

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/cpp_linter/clang_tools/clang_tidy.py

@@ -92,7 +92,10 @@ class TidyNotification:
             check_name_parts = self.diagnostic.split("-", maxsplit=2)
             assert len(check_name_parts) > 2, "diagnostic name malformed"
             return link + "clang-analyzer/{}.html)".format(check_name_parts[2])
-        return link + "{}/{}.html)".format(*self.diagnostic.split("-", maxsplit=1))
+        diag_split = self.diagnostic.split("-", maxsplit=1)
+        if len(diag_split) < 2 or not all(diag_split):
+            return self.diagnostic
+        return link + "{}/{}.html)".format(*diag_split)
 
     def __repr__(self) -> str:
         return (
@@ -248,7 +251,8 @@ def run_clang_tidy(
         extra_args = extra_args[0].split()
     for extra_arg in extra_args:
         arg = extra_arg.strip('"')
-        cmds.append(f"--extra-arg={arg}")
+        if arg:  # avoid adding empty arg values
+            cmds.append(f"--extra-arg={arg}")
     if tidy_review:
         # clang-tidy overwrites the file contents when applying fixes.
         # create a cache of original contents
@@ -289,16 +293,21 @@ def parse_tidy_output(
         note_match = re.match(NOTE_HEADER, line)
         fixed_match = re.match(FIXED_NOTE, line)
         if note_match is not None:
-            notification = TidyNotification(
-                cast(
-                    Tuple[str, Union[int, str], Union[int, str], str, str, str],
-                    note_match.groups(),
-                ),
-                database,
-            )
-            tidy_notes.append(notification)
-            # begin capturing subsequent lines as part of notification details
-            found_fix = False
+            # Sometimes clang-tidy uses square brackets to enclose additional context
+            # about the diagnostic rationale. For example: '[with auto = typename ...]'
+            # We need to ignore such cases as they do not start a diagnostic report.
+            diagnostic_name = note_match.group(6)
+            if " " not in diagnostic_name and "-" in diagnostic_name:
+                notification = TidyNotification(
+                    cast(
+                        Tuple[str, Union[int, str], Union[int, str], str, str, str],
+                        note_match.groups(),
+                    ),
+                    database,
+                )
+                tidy_notes.append(notification)
+                # begin capturing subsequent lines as part of notification details
+                found_fix = False
         elif fixed_match is not None and notification is not None:
             notification.applied_fixes.add(int(fixed_match.group(1)))
             # suspend capturing subsequent lines as they are not needed

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/cpp_linter/common_fs/__init__.py

@@ -67,10 +67,10 @@ class FileObj:
             if not i:
                 result.append([n])
             elif n - 1 != numbers[i - 1]:
-                result[-1].append(numbers[i - 1] + 1)
+                result[-1].append(numbers[i - 1])
                 result.append([n])
             if i == len(numbers) - 1:
-                result[-1].append(n + 1)
+                result[-1].append(n)
         return result
 
     def range_of_changed_lines(

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/cpp_linter/rest_api/github_api.py

@@ -57,7 +57,10 @@ class GithubApiClient(RestApiClient):
         #: The HEAD commit's SHA
         self.sha = environ.get("GITHUB_SHA", "")
         #: A flag that describes if debug logs are enabled.
-        self.debug_enabled = environ.get("ACTIONS_STEP_DEBUG", "") == "true"
+        self.debug_enabled = (
+            environ.get("ACTIONS_STEP_DEBUG", "") == "true"
+            or environ.get("ACTIONS_RUNNER_DEBUG", "") == "true"
+        )
 
         #: The pull request number for the event (if applicable).
         self.pull_request = -1

{cpp_linter-1.11.3 → cpp_linter-1.11.4/cpp_linter.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cpp-linter
-Version: 1.11.3
+Version: 1.11.4
 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.11.3 → cpp_linter-1.11.4}/cpp_linter.egg-info/SOURCES.txt

@@ -70,6 +70,7 @@ tests/test_cli_args.py
 tests/test_comment_length.py
 tests/test_git_str.py
 tests/test_misc.py
+tests/test_parse_clang_tidy_stdout.py
 tests/test_rate_limits.py
 tests/capture_tools_output/test_database_path.py
 tests/capture_tools_output/test_tools_output.py

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/cspell.config.yml

@@ -14,6 +14,7 @@ words:
   - consts
   - cppcoreguidelines
   - cstdio
+  - declval
   - docutils
   - endgroup
   - Fixit

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/tests/capture_tools_output/chocolate-doom/chocolate-doom/67715d6e2725322e6132e9ff99b9a2a3f3b10c83/expected-result_0.json

@@ -11,11 +11,11 @@
       "lines_added": [
         [
           53,
-          54
+          53
         ],
         [
           55,
-          56
+          55
         ]
       ]
     }
@@ -36,11 +36,11 @@
       "lines_added": [
         [
           36,
-          40
+          39
         ],
         [
           518,
-          520
+          519
         ]
       ]
     }
@@ -57,11 +57,11 @@
       "lines_added": [
         [
           59,
-          60
+          59
         ],
         [
           61,
-          62
+          61
         ]
       ]
     }
@@ -82,11 +82,11 @@
       "lines_added": [
         [
           27,
-          31
+          30
         ],
         [
           255,
-          257
+          256
         ]
       ]
     }
@@ -111,11 +111,11 @@
       "lines_added": [
         [
           47,
-          54
+          53
         ],
         [
           1384,
-          1464
+          1463
         ]
       ]
     }
@@ -140,11 +140,11 @@
       "lines_added": [
         [
           128,
-          147
+          146
         ],
         [
           597,
-          599
+          598
         ]
       ]
     }
@@ -169,11 +169,11 @@
       "lines_added": [
         [
           44,
-          59
+          58
         ],
         [
           1144,
-          1146
+          1145
         ]
       ]
     }
@@ -194,19 +194,19 @@
       "lines_added": [
         [
           102,
-          103
+          102
         ],
         [
           104,
-          105
+          104
         ],
         [
           113,
-          114
+          113
         ],
         [
           115,
-          116
+          115
         ]
       ]
     }
@@ -227,11 +227,11 @@
       "lines_added": [
         [
           36,
-          40
+          39
         ],
         [
           383,
-          442
+          441
         ]
       ]
     }

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/tests/capture_tools_output/chocolate-doom/chocolate-doom/67715d6e2725322e6132e9ff99b9a2a3f3b10c83/expected-result_1.json

@@ -11,11 +11,11 @@
       "lines_added": [
         [
           53,
-          54
+          53
         ],
         [
           55,
-          56
+          55
         ]
       ]
     }
@@ -36,11 +36,11 @@
       "lines_added": [
         [
           36,
-          40
+          39
         ],
         [
           518,
-          520
+          519
         ]
       ]
     }
@@ -57,11 +57,11 @@
       "lines_added": [
         [
           59,
-          60
+          59
         ],
         [
           61,
-          62
+          61
         ]
       ]
     }
@@ -82,11 +82,11 @@
       "lines_added": [
         [
           27,
-          31
+          30
         ],
         [
           255,
-          257
+          256
         ]
       ]
     }
@@ -111,11 +111,11 @@
       "lines_added": [
         [
           47,
-          54
+          53
         ],
         [
           1384,
-          1464
+          1463
         ]
       ]
     }
@@ -140,11 +140,11 @@
       "lines_added": [
         [
           128,
-          147
+          146
         ],
         [
           597,
-          599
+          598
         ]
       ]
     }
@@ -169,11 +169,11 @@
       "lines_added": [
         [
           44,
-          59
+          58
         ],
         [
           1144,
-          1146
+          1145
         ]
       ]
     }
@@ -194,19 +194,19 @@
       "lines_added": [
         [
           102,
-          103
+          102
         ],
         [
           104,
-          105
+          104
         ],
         [
           113,
-          114
+          113
         ],
         [
           115,
-          116
+          115
         ]
       ]
     }
@@ -227,11 +227,11 @@
       "lines_added": [
         [
           36,
-          40
+          39
         ],
         [
           383,
-          442
+          441
         ]
       ]
     }

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/tests/capture_tools_output/chocolate-doom/chocolate-doom/67715d6e2725322e6132e9ff99b9a2a3f3b10c83/expected-result_2.json

@@ -11,11 +11,11 @@
       "lines_added": [
         [
           53,
-          54
+          53
         ],
         [
           55,
-          56
+          55
         ]
       ]
     }
@@ -36,11 +36,11 @@
       "lines_added": [
         [
           36,
-          40
+          39
         ],
         [
           518,
-          520
+          519
         ]
       ]
     }
@@ -57,11 +57,11 @@
       "lines_added": [
         [
           59,
-          60
+          59
         ],
         [
           61,
-          62
+          61
         ]
       ]
     }
@@ -82,11 +82,11 @@
       "lines_added": [
         [
           27,
-          31
+          30
         ],
         [
           255,
-          257
+          256
         ]
       ]
     }
@@ -111,11 +111,11 @@
       "lines_added": [
         [
           47,
-          54
+          53
         ],
         [
           1384,
-          1464
+          1463
         ]
       ]
     }
@@ -140,11 +140,11 @@
       "lines_added": [
         [
           128,
-          147
+          146
         ],
         [
           597,
-          599
+          598
         ]
       ]
     }
@@ -169,11 +169,11 @@
       "lines_added": [
         [
           44,
-          59
+          58
         ],
         [
           1144,
-          1146
+          1145
         ]
       ]
     }
@@ -194,19 +194,19 @@
       "lines_added": [
         [
           102,
-          103
+          102
         ],
         [
           104,
-          105
+          104
         ],
         [
           113,
-          114
+          113
         ],
         [
           115,
-          116
+          115
         ]
       ]
     }
@@ -227,11 +227,11 @@
       "lines_added": [
         [
           36,
-          40
+          39
         ],
         [
           383,
-          442
+          441
         ]
       ]
     }

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/tests/capture_tools_output/chocolate-doom/chocolate-doom/71091562db5b0e7853d08ffa2f110af49cc3bc0d/expected-result_0.json

@@ -51,7 +51,7 @@
       "lines_added": [
         [
           197,
-          214
+          213
         ]
       ]
     }

{cpp_linter-1.11.3 → cpp_linter-1.11.4}/tests/capture_tools_output/chocolate-doom/chocolate-doom/71091562db5b0e7853d08ffa2f110af49cc3bc0d/expected-result_1.json

@@ -51,7 +51,7 @@
       "lines_added": [
         [
           197,
-          214
+          213
         ]
       ]
     }