ci: Separarate py/lint workflows (#2814)

Signed-off-by: Ryan Northey <[email protected]>

Author: phlax

.github/workflows/lint.yml

@@ -0,0 +1,24 @@
+name: Lint
+
+on:
+  push:
+    branches:
+    - "main"
+  pull_request:
+    branches:
+    - "main"
+
+permissions:
+  contents: read
+
+jobs:
+  lint:
+    runs-on: ubuntu-24.04
+    steps:
+    - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683  # v4.2.2
+    - uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065  # v5.6.0
+      with:
+        python-version: "3.13"
+    - run: pip install --require-hashes -r .github/workflows/requirements.txt
+    - run: |
+        envoy.code.check . -c glint shellcheck yamllint  -x ".*/dist/.*" -x "rust/glint/tests/fixtures/.*.txt"

.github/workflows/ci.yml renamed to .github/workflows/py.yml

@@ -1,4 +1,4 @@
-name: CI
+name: Python
 
 on:
   push:
@@ -10,7 +10,7 @@ on:
     - "**/BUILD"
     - "**/BUILD.pyi"
     - "**/BUILD.tools"
-    - ".github/workflows/ci.yml"
+    - ".github/workflows/py.yml"
     - "pants.toml"
     - "pants.ci.toml"
     - ".coveragerc"
@@ -39,7 +39,7 @@ on:
     - "**/BUILD"
     - "**/BUILD.pyi"
     - "**/BUILD.tools"
-    - ".github/workflows/ci.yml"
+    - ".github/workflows/py.yml"
     - "pants.toml"
     - "pants.ci.toml"
     - ".coveragerc"
@@ -103,17 +103,6 @@ jobs:
     - name: Run pants lint
       run: "pants --colors lint ::"
 
-  lint-envoy:
-    runs-on: ubuntu-24.04
-    steps:
-    - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683  # v4.2.2
-    - uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065  # v5.6.0
-      with:
-        python-version: "3.11"
-    - run: pip install --require-hashes -r .github/workflows/requirements.txt
-    - run: |
-        envoy.code.check . -c glint shellcheck yamllint  -x ".*/dist/.*"
-
   typecheck:
     runs-on: ubuntu-24.04
     steps:

rust/glint/tests/fixtures/README.md

@@ -4,7 +4,7 @@ This directory contains test files and their expected outputs.
 
 ## Test Files:
 - `bad.txt` - 50 lines with all types of issues (trailing whitespace, mixed indentation, no final newline)
-- `bad2.txt` - 7 lines with all types of issues  
+- `bad2.txt` - 7 lines with all types of issues
 - `bad3.txt` - 7 lines with only trailing whitespace issues
 - `good.txt` - Clean file with no issues
 

rust/glint/tests/integration_test.rs

@@ -8,46 +8,46 @@ fn run_glint_test(fixture_name: &str, expect_success: bool) {
     cmd.arg("run");
     cmd.arg("--quiet");
     cmd.arg("--");
-    
+
     // Get the path to the test fixture
     let fixture_path = PathBuf::from(env!("CARGO_MANIFEST_DIR"))
         .join("tests")
         .join("fixtures")
         .join(fixture_name);
-    
+
     cmd.arg(&fixture_path);
-    
+
     let output = cmd.output().expect("Failed to execute glint");
-    
+
     // Check exit code
     if expect_success {
-        assert!(output.status.success(), 
-            "Expected glint to exit with zero status for {}, got {:?}", 
+        assert!(output.status.success(),
+            "Expected glint to exit with zero status for {}, got {:?}",
             fixture_name, output.status.code());
     } else {
-        assert!(!output.status.success(), 
+        assert!(!output.status.success(),
             "Expected glint to exit with non-zero status for {}", fixture_name);
-        assert_eq!(output.status.code(), Some(1), 
+        assert_eq!(output.status.code(), Some(1),
             "Expected exit code 1 for {}", fixture_name);
     }
-    
+
     // Parse the actual JSON output
     let stdout = String::from_utf8_lossy(&output.stdout);
     let actual_json: Value = serde_json::from_str(&stdout)
         .expect(&format!("Failed to parse JSON output for {}: {}", fixture_name, stdout));
-    
+
     // Load expected JSON
     let expected_path = PathBuf::from(env!("CARGO_MANIFEST_DIR"))
         .join("tests")
         .join("fixtures")
         .join(format!("{}.expected.json", fixture_name));
-    
+
     let expected_content = fs::read_to_string(&expected_path)
         .expect(&format!("Failed to read expected file: {:?}", expected_path));
-    
+
     let mut expected_json: Value = serde_json::from_str(&expected_content)
         .expect(&format!("Failed to parse expected JSON for {}", fixture_name));
-    
+
     // Normalize file paths in expected JSON to match actual output
     if let Some(files) = expected_json.get_mut("files").and_then(|f| f.as_object_mut()) {
         let mut normalized_files = serde_json::Map::new();
@@ -61,11 +61,11 @@ fn run_glint_test(fixture_name: &str, expect_success: bool) {
         }
         *files = normalized_files;
     }
-    
+
     // Compare JSON
-    assert_eq!(actual_json, expected_json, 
-        "JSON output mismatch for {}.\nActual:\n{}\nExpected:\n{}", 
-        fixture_name, 
+    assert_eq!(actual_json, expected_json,
+        "JSON output mismatch for {}.\nActual:\n{}\nExpected:\n{}",
+        fixture_name,
         serde_json::to_string_pretty(&actual_json).unwrap(),
         serde_json::to_string_pretty(&expected_json).unwrap());
 }