Skip to content

Commit b409221

Browse files
Mlaz-codeclaude
andcommitted
ci: invoke ruff + pyright in test.yml + publish.yml
Adds two new steps to the existing test matrix (Python 3.10-3.13) and mirrors them into publish.yml's gate test job. pyproject.toml has had [tool.ruff] and [tool.pyright] configs since the repo was created; now they actually run. Lint + type-check are cheap compared to the test suite (<5s each) and catch the class of issue the previous commit fixed before it can land on main. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 90c2333 commit b409221

2 files changed

Lines changed: 21 additions & 1 deletion

File tree

.github/workflows/publish.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,15 @@ jobs:
3030
python-version: ${{ matrix.python-version }}
3131

3232
- name: Install dependencies
33-
run: pip install -e ".[test,pandas]"
33+
run: |
34+
pip install -e ".[test,pandas]"
35+
pip install ruff pyright
36+
37+
- name: Ruff check
38+
run: ruff check
39+
40+
- name: Pyright
41+
run: pyright
3442

3543
- name: Run tests
3644
run: pytest -v

.github/workflows/test.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,18 @@ jobs:
2323
- name: Install dependencies
2424
run: |
2525
pip install -e ".[test,pandas]"
26+
pip install ruff pyright
27+
28+
- name: Ruff check
29+
# [tool.ruff] config is already in pyproject.toml — rules E, F,
30+
# I, UP. This step enforces them; before today they were only
31+
# suggestions.
32+
run: ruff check
33+
34+
- name: Pyright
35+
# Same story — [tool.pyright] was set to standard mode in
36+
# pyproject.toml but never invoked in CI.
37+
run: pyright
2638

2739
- name: Run tests
2840
run: pytest -v

0 commit comments

Comments
 (0)