Skip to content

Commit f6ef9c8

Browse files
committed
ci: enforce pre-commit in quality workflow
1 parent 15ae25f commit f6ef9c8

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

.github/workflows/test.yml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,8 @@ jobs:
2828
python -m pip install --upgrade pip
2929
pip install -e ".[all]"
3030
31-
- name: Run linting
32-
run: ruff check src/ tests/
33-
34-
- name: Check formatting
35-
run: ruff format --check src/ tests/
31+
- name: Run pre-commit checks
32+
run: pre-commit run --all-files
3633

3734
- name: Run type checking
3835
run: mypy src/notebooklm --ignore-missing-imports

docs/development.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,8 @@ src/notebooklm/
111111
uv run pre-commit install
112112
```
113113

114+
CI runs the same lint gate with `uv run pre-commit run --all-files`, so local hook results should match the `quality` job.
115+
114116
2. **Authenticate:**
115117
```bash
116118
notebooklm login

docs/releasing.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -136,6 +136,7 @@ Proceed with release preparation?
136136
```bash
137137
uv run pre-commit run --all-files && uv run mypy src/notebooklm --ignore-missing-imports && uv run pytest
138138
```
139+
- [ ] Ensure CI runs the same lint gate (`pre-commit run --all-files`) as local release prep
139140
- [ ] Fix any issues before proceeding
140141

141142
### Commit

0 commit comments

Comments
 (0)