Skip to content

Commit 2c368a3

Browse files
thomas-manginclaude
andcommitted
Change: Add qa/ folder to ruff check in CI and test_everything
- Update linting.yml: ruff check . → ruff check src qa tests - Update qa/bin/test_everything: ruff check src tests → ruff check src qa tests - Fix ruff errors in qa/scripts: - test_api_commands.py: bare except → except Exception - test_no_neighbor.py: remove unused signal import, bare except → except Exception Now CI and local tests explicitly check same folders (src, qa, tests). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent ec867bd commit 2c368a3

File tree

4 files changed

+5
-6
lines changed

4 files changed

+5
-6
lines changed

.github/workflows/linting.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,4 +35,4 @@ jobs:
3535
run: |
3636
python --version
3737
ruff --version
38-
ruff check . --output-format=full --statistics
38+
ruff check src qa tests --output-format=full --statistics

qa/bin/test_everything

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,9 +129,9 @@ def test_ruff_format() -> Tuple[bool, int]:
129129
def test_ruff_check() -> Tuple[bool, int]:
130130
"""Test 2: Ruff check"""
131131
print_header("TEST 2/6: Ruff Check")
132-
print_info("Running: ruff check src tests")
132+
print_info("Running: ruff check src qa tests")
133133

134-
returncode, stdout, stderr = run_command("ruff check src tests")
134+
returncode, stdout, stderr = run_command("ruff check src qa tests")
135135

136136
if returncode != 0:
137137
print_error("Ruff check failed")

qa/scripts/test_api_commands.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ def test_api_commands():
101101
try:
102102
proc.kill()
103103
proc.wait()
104-
except:
104+
except Exception:
105105
pass
106106
return False
107107

qa/scripts/test_no_neighbor.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
import sys
1616
import os
1717
import time
18-
import signal
1918

2019
# Paths relative to repository root
2120
SCRIPT_DIR = os.path.dirname(os.path.abspath(__file__))
@@ -109,7 +108,7 @@ def test_no_neighbor():
109108
try:
110109
proc.kill()
111110
proc.wait()
112-
except:
111+
except Exception:
113112
pass
114113
return False
115114

0 commit comments

Comments
 (0)