Move quality check to separate workflow file and fix error msg

Signed-off-by: Fynn Schmitt-Ulms <[email protected]>

Author: fynnsu

.github/workflows/quality-check.yaml

@@ -0,0 +1,27 @@
+name: Quality Checks
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  quality-check:
+    runs-on: ubuntu-24.04
+    steps:
+        - uses: actions/setup-python@v5
+          with:
+            python-version: '3.10'
+        - uses: actions/checkout@v4
+          with:
+            fetch-depth: 0
+            fetch-tags: true
+        - name: Set Env
+          run: |
+            pip3 install --upgrade pip && pip3 install --upgrade setuptools
+        - name: "⚙️ Install dependencies"
+          run: pip3 install .[dev]
+        - name: "🧹 Running quality checks"
+          run: make quality

.github/workflows/test-check.yaml

@@ -27,20 +27,3 @@ jobs:
         - name: "🔬 Running tests"
           run: make test
 
-  python-style:
-    runs-on: ubuntu-24.04
-    steps:
-        - uses: actions/setup-python@v5
-          with:
-            python-version: '3.10'
-        - uses: actions/checkout@v4
-          with:
-            fetch-depth: 0
-            fetch-tags: true
-        - name: Set Env
-          run: |
-            pip3 install --upgrade pip && pip3 install --upgrade setuptools
-        - name: "⚙️ Install dependencies"
-          run: pip3 install .[dev]
-        - name: "🔬 Running quality checks"
-          run: make quality

src/compressed_tensors/quantization/quant_args.py

@@ -211,21 +211,25 @@ def validate_group(cls, value) -> Union[int, None]:
     def validate_block_structure(cls, value) -> Optional[List[int]]:
         if value is None:
             return value
-        invalid_block_structure_msg = (
-            f"Invalid block_structure '{value}'. Must be a list of two ints"
-            " [rows, cols]."
-        )
         # For backward compatibility, allow string format "2x4", "8x16", etc.
         if isinstance(value, str):
             try:
                 return [int(x) for x in value.split("x")]
             except Exception:
-                raise ValueError(invalid_block_structure_msg)
+                raise ValueError(
+                    f"Invalid block_structure '{value}'. Must be a list of ints "
+                    "[rows, cols]."
+                )
         if isinstance(value, (list, tuple)):
             if len(value) != 2 or not all(isinstance(v, int) for v in value):
-                raise ValueError(invalid_block_structure_msg)
+                raise ValueError(
+                    f"Invalid block_structure '{value}'. Must be a list of ints "
+                    "[rows, cols]."
+                )
             return list(value)
-        raise ValueError(invalid_block_structure_msg)
+        raise ValueError(
+            f"Invalid block_structure '{value}'. Must be a list of ints [rows, cols]."
+        )
 
     @field_validator("strategy", mode="before")
     def validate_strategy(cls, value) -> Union[QuantizationStrategy, None]: