build(deps): bump the cargo group across 1 directory with 6 updates (#176)

* build(deps): bump the cargo group across 1 directory with 6 updates

Bumps the cargo group with 6 updates in the / directory:

| Package | From | To |
| --- | --- | --- |
| [clap](https://github.com/clap-rs/clap) | `4.5.45` | `4.5.46` |
| [fast-glob](https://github.com/oxc-project/fast-glob) | `0.4.5` | `1.0.0` |
| [quick-xml](https://github.com/tafia/quick-xml) | `0.38.1` | `0.38.3` |
| [regex](https://github.com/rust-lang/regex) | `1.11.1` | `1.11.2` |
| [tempfile](https://github.com/Stebalien/tempfile) | `3.20.0` | `3.21.0` |
| [pyo3](https://github.com/pyo3/pyo3) | `0.25.1` | `0.26.0` |



Updates `clap` from 4.5.45 to 4.5.46
- [Release notes](https://github.com/clap-rs/clap/releases)
- [Changelog](https://github.com/clap-rs/clap/blob/master/CHANGELOG.md)
- [Commits](clap-rs/clap@clap_complete-v4.5.45...clap_complete-v4.5.46)

Updates `fast-glob` from 0.4.5 to 1.0.0
- [Release notes](https://github.com/oxc-project/fast-glob/releases)
- [Changelog](https://github.com/oxc-project/fast-glob/blob/main/CHANGELOG.md)
- [Commits](oxc-project/fast-glob@v0.4.5...v1.0.0)

Updates `quick-xml` from 0.38.1 to 0.38.3
- [Release notes](https://github.com/tafia/quick-xml/releases)
- [Changelog](https://github.com/tafia/quick-xml/blob/master/Changelog.md)
- [Commits](tafia/quick-xml@v0.38.1...v0.38.3)

Updates `regex` from 1.11.1 to 1.11.2
- [Release notes](https://github.com/rust-lang/regex/releases)
- [Changelog](https://github.com/rust-lang/regex/blob/master/CHANGELOG.md)
- [Commits](rust-lang/regex@1.11.1...1.11.2)

Updates `tempfile` from 3.20.0 to 3.21.0
- [Changelog](https://github.com/Stebalien/tempfile/blob/master/CHANGELOG.md)
- [Commits](Stebalien/tempfile@v3.20.0...v3.21.0)

Updates `pyo3` from 0.25.1 to 0.26.0
- [Release notes](https://github.com/pyo3/pyo3/releases)
- [Changelog](https://github.com/PyO3/pyo3/blob/main/CHANGELOG.md)
- [Commits](PyO3/pyo3@v0.25.1...v0.26.0)

---
updated-dependencies:
- dependency-name: clap
  dependency-version: 4.5.46
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: cargo
- dependency-name: fast-glob
  dependency-version: 1.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: cargo
- dependency-name: quick-xml
  dependency-version: 0.38.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: cargo
- dependency-name: regex
  dependency-version: 1.11.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: cargo
- dependency-name: tempfile
  dependency-version: 3.21.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: cargo
- dependency-name: pyo3
  dependency-version: 0.26.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: cargo
...

Signed-off-by: dependabot[bot] <support@github.com>

* adjust test CI matrix for supported clang versions

* use llvm-cov to generate HTML coverage report

* fix clippy lints

and make CI fail on lint warnings

* auto-cancel benchmark run when superseded

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Brendan <2bndy5@gmail.com>

Author: dependabot[bot]

.github/install-clang-action/action.yml

@@ -33,6 +33,7 @@ runs:
       shell: bash
       continue-on-error: true
       run: |
+        brew update
         brew install llvm@${{ inputs.version }}
         ln -s "$(brew --prefix llvm@${{ inputs.version }})/bin/clang-format" "/usr/local/bin/clang-format-${{ inputs.version }}"
         ln -s "$(brew --prefix llvm@${{ inputs.version }})/bin/clang-tidy" "/usr/local/bin/clang-tidy-${{ inputs.version }}"

.github/workflows/benchmark.yml

@@ -24,6 +24,12 @@ on:
   # performance analysis in order to generate initial data.
   workflow_dispatch:
 
+# This CI workflow can take up to 2 hours.
+# This setting will auto-cancel a old run if a new run is started.
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   benchmark:
     runs-on: ubuntu-latest

.github/workflows/pre-commit-hooks.yml

@@ -26,7 +26,7 @@ jobs:
           path: ~/.cargo
           key: ${{ runner.os }}-cargo-${{ hashFiles('Cargo.lock') }}
       - name: cargo clippy
-        run: cargo clippy
+        run: cargo clippy -- -D warnings
       - name: cargo fmt
         run: cargo fmt --check
 

.github/workflows/run-dev-tests.yml

@@ -52,9 +52,6 @@ jobs:
         with:
           tool: cargo-nextest,cargo-llvm-cov,cargo-binstall
 
-      - name: Install llvm-cov-pretty (HTML report generator)
-        run: cargo binstall -y llvm-cov-pretty
-
       - uses: actions/setup-python@v5
         with:
           python-version: 3.x
@@ -84,18 +81,6 @@ jobs:
       - if: runner.os == 'Linux'
         run: sudo apt-get update
 
-      - name: Install clang v8
-        if: runner.os == 'Linux'
-        uses: ./.github/install-clang-action
-        with:
-          version: '8'
-
-      - name: Collect Coverage for clang v8
-        if: runner.os == 'Linux'
-        env:
-          CLANG_VERSION: '8'
-        run: uvx nox -s test -- --profile ci
-
       - name: Install clang v9
         uses: ./.github/install-clang-action
         with:
@@ -194,21 +179,55 @@ jobs:
       - name: Collect Coverage for clang v18
         env:
           CLANG_VERSION: '18'
+        run: uvx nox -s test -- --profile ci
+
+      - name: Install clang v19
+        if: runner.os == 'Linux'
+        uses: ./.github/install-clang-action
+        with:
+          version: '19'
+
+      - name: Collect Coverage for clang v19
+        if: runner.os == 'Linux'
+        env:
+          CLANG_VERSION: '19'
+        run: uvx nox -s test -- --profile ci
+
+      - name: Install clang v20
+        if: runner.os == 'Linux'
+        uses: ./.github/install-clang-action
+        with:
+          version: '20'
+
+      - name: Collect Coverage for clang v20
+        if: runner.os == 'Linux'
+        env:
+          CLANG_VERSION: '20'
+        run: uvx nox -s test -- --profile ci
+
+      - name: Install clang v21
+        if: runner.os == 'Linux'
+        uses: ./.github/install-clang-action
+        with:
+          version: '21'
+
+      - name: Collect Coverage for clang v21
+        if: runner.os == 'Linux'
+        env:
+          CLANG_VERSION: '21'
         run: uvx nox -s test -- --profile all
 
       - name: Generate Coverage HTML report
-        run: uvx nox -s pretty-cov
+        run: uvx nox -s llvm-cov
 
       - name: Upload coverage data
         uses: actions/upload-artifact@v4
         with:
           name: HTML_report-${{ runner.os }}
-          path: target/llvm-cov-pretty
+          path: target/llvm-cov/html
 
       - name: Generate Coverage lcov report
-        run: |
-          rm coverage.json
-          uvx nox -s lcov
+        run: uvx nox -s lcov
 
       - uses: codecov/codecov-action@v5
         with:

Cargo.lock

@@ -17,7 +17,7 @@ crate-type = ["cdylib"]
 bench = false
 
 [dependencies]
-pyo3 = { version = "0.25.1", features = ["extension-module"] }
+pyo3 = { version = "0.26.0", features = ["extension-module"] }
 cpp-linter = { path = "../../cpp-linter" }
 tokio = "1.47.1"
 

bindings/python/Cargo.toml

@@ -16,17 +16,17 @@ license.workspace = true
 [dependencies]
 anyhow = "1.0.99"
 chrono = "0.4.41"
-clap = "4.5.45"
+clap = "4.5.46"
 colored = "3.0.0"
-fast-glob = "0.4.5"
+fast-glob = "1.0.0"
 futures = "0.3.31"
 git2 = "0.20.2"
 lenient_semver = "0.4.2"
 log = { version = "0.4.27", features = ["std"] }
 openssl = { version = "0.10", features = ["vendored"], optional = true }
 openssl-probe = { version = "0.1", optional = true }
-quick-xml = {version = "0.38.1", features = ["serialize"]}
-regex = "1.11.1"
+quick-xml = {version = "0.38.3", features = ["serialize"]}
+regex = "1.11.2"
 reqwest = "0.12.23"
 semver = "1.0.26"
 serde = { version = "1.0.219", features = ["derive"] }
@@ -39,7 +39,7 @@ which = "8.0.0"
 [dev-dependencies]
 criterion = { version = "3.0.5", package = "codspeed-criterion-compat", features=["async_tokio"] }
 mockito = "1.7.0"
-tempfile = "3.20.0"
+tempfile = "3.21.0"
 
 [features]
 openssl-vendored = ["dep:openssl", "dep:openssl-probe"]

cpp-linter/Cargo.toml

@@ -65,9 +65,9 @@ pub fn get_clang_tool_exe(name: &str, version: &str) -> Result<PathBuf> {
             // On Unix systems, this block is not likely reached. Typically, installing clang
             // will produce a symlink to the executable with the major version appended to the
             // name.
-            return Ok(cmd);
+            Ok(cmd)
         } else {
-            return Err(anyhow!("Could not find clang tool by name and version"));
+            Err(anyhow!("Could not find clang tool by name and version"))
         }
     } else {
         // `version` specified is not a semantic version; treat as path/to/bin

cpp-linter/src/clang_tools/mod.rs

@@ -49,7 +49,7 @@ fn get_sha(repo: &Repository, depth: Option<u32>) -> Result<git2::Object<'_>, Er
 /// If there are files staged for a commit, then the resulting [`Diff`] will describe
 /// the staged changes. However, if there are no staged changes, then the last commit's
 /// [`Diff`] is returned.
-pub fn get_diff(repo: &Repository) -> Result<git2::Diff> {
+pub fn get_diff(repo: &'_ Repository) -> Result<git2::Diff<'_>> {
     let head = get_sha(repo, None).unwrap().peel_to_tree().unwrap();
     let mut has_staged_files = false;
     for entry in repo.statuses(None).unwrap().iter() {

cpp-linter/src/git.rs

@@ -10,7 +10,7 @@ license.workspace = true
 
 [dependencies]
 cpp-linter = { path = "../cpp-linter" }
-pyo3 = "0.25.1"
+pyo3 = "0.26.0"
 
 [lib]
 name = "cli_gen"