Skip to content

Commit 821758c

Browse files
wmaynerclaude
andcommitted
Separate lint and test CI workflows
Move type checking and pre-commit hooks to a dedicated lint workflow that runs once, rather than repeating pyright across the full OS/Python test matrix. This provides faster feedback and proper enforcement. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 65a1752 commit 821758c

File tree

2 files changed

+35
-4
lines changed

2 files changed

+35
-4
lines changed

.github/workflows/lint.yml

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
name: Lint
2+
3+
on:
4+
push:
5+
branches: [develop, main, master]
6+
pull_request:
7+
branches: [develop, main, master]
8+
9+
jobs:
10+
lint:
11+
runs-on: ubuntu-latest
12+
13+
steps:
14+
- name: Check out repository
15+
uses: actions/checkout@v4
16+
17+
- name: Set up uv
18+
uses: astral-sh/setup-uv@v5
19+
with:
20+
enable-cache: true
21+
cache-dependency-glob: "uv.lock"
22+
23+
- name: Set up Python
24+
run: uv python install 3.12
25+
26+
- name: Install dependencies
27+
run: uv sync --python 3.12 --group dev --all-extras
28+
29+
- name: Run pre-commit hooks
30+
uses: pre-commit/action@v3.0.1
31+
env:
32+
SKIP: pyright
33+
34+
- name: Run type checking
35+
run: uv run pyright pyphi

.github/workflows/test.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,6 @@ jobs:
3131
- name: Install dependencies
3232
run: uv sync --python ${{ matrix.python-version }} --group dev --all-extras
3333

34-
- name: Run type checking
35-
run: uv run pyright pyphi
36-
continue-on-error: true
37-
3834
- name: Run tests
3935
run: uv run pytest test/ --tb=short -v --ignore=test/test_parallel.py --ignore=test/test_parallel2.py
4036
continue-on-error: ${{ matrix.os == 'windows-latest' }}

0 commit comments

Comments
 (0)