fix: add required=True and update test.yml

Author: shenxianpeng

.github/workflows/test.yml

@@ -147,39 +147,48 @@ jobs:
             clang-apply-replacements-${{ matrix.version }} --version
           fi
 
-      - name: Install clang-tools wheels
-        if: matrix.version >= 13 # Only versions 13 and above have wheels
-        run: |
-          clang-tools-wheel --tool clang-format --version ${{ matrix.version }}
-          clang-tools-wheel --tool clang-tidy --version ${{ matrix.version }}
-
-      - name: Check clang-tools wheels
-        if: matrix.version >= 13
+      - name: Install and check clang-format wheels
         shell: bash
         run: |
           set -e
-          echo "Checking clang-format and clang-tidy versions..."
+          clang-tools-wheel --tool clang-format --version ${{ matrix.version }}
+
+          echo "Checking clang-format versions..."
           clang-format --version
-          clang-tidy --version
 
           # Verify versions contain expected string
           clang_format_version=$(clang-format --version)
-          clang_tidy_version=$(clang-tidy --version)
 
           echo "clang-format version: $clang_format_version"
-          echo "clang-tidy version: $clang_tidy_version"
 
           if ! echo "$clang_format_version" | grep -q "version ${{ matrix.version }}"; then
             echo "❌ Unexpected clang-format version!"
             exit 1
           fi
 
+          echo "✅ clang-format Versions are correct."
+
+      - name: Install and check clang-tidy wheels
+        if: matrix.version >= 13 # Only versions 13 and above have wheels
+        shell: bash
+        run: |
+          set -e
+          clang-tools-wheel --tool clang-tidy --version ${{ matrix.version }}
+
+          echo "Checking clang-tidy versions..."
+          clang-tidy --version
+
+          # Verify versions contain expected string
+          clang_tidy_version=$(clang-tidy --version)
+
+          echo "clang-tidy version: $clang_tidy_version"
+
           if ! echo "$clang_tidy_version" | grep -q "version ${{ matrix.version }}"; then
             echo "❌ Unexpected clang-tidy version!"
             exit 1
           fi
 
-          echo "✅ Versions are correct."
+          echo "✅ clang-tidy Versions are correct."
 
   docs:
     uses: cpp-linter/.github/.github/workflows/sphinx.yml@main

clang_tools/wheel.py

@@ -8,6 +8,7 @@ def get_parser() -> ArgumentParser:
     parser.add_argument(
         "--tool",
         default="clang-format",
+        required=True,
         choices=["clang-format", "clang-tidy"],
         help="Tool to install (clang-format or clang-tidy)",
     )