Skip to content

Commit fd20b93

Browse files
committed
Fix CI: separate docs deps and add fail-fast: false
The poetry install step failed on Python 3.12 because pydoc-markdown 2.1.3 (via yapf) is incompatible with 3.12. - Move mkdocs, pydoc-markdown, mkdocs-material to optional "docs" dependency group so they aren't installed during CI test runs - Add fail-fast: false so one matrix failure doesn't cancel all jobs - Use --without docs flag in poetry install https://claude.ai/code/session_017D3uGziwAhF88Zivc51jK5
1 parent 76794eb commit fd20b93

3 files changed

Lines changed: 33 additions & 27 deletions

File tree

.github/workflows/tests.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ jobs:
88
runs-on: ubuntu-latest
99

1010
strategy:
11+
fail-fast: false
1112
matrix:
1213
python-version: ["3.9", "3.10", "3.11", "3.12"]
1314

@@ -39,7 +40,7 @@ jobs:
3940
key: venv-${{ runner.os }}-${{ steps.setup-python.outputs.python-version }}-${{ hashFiles('**/poetry.lock') }}
4041
- name: Install dependencies (if no cache)
4142
if: steps.cached-poetry-dependencies.outputs.cache-hit != 'true'
42-
run: poetry install --no-interaction
43+
run: poetry install --no-interaction --without docs
4344

4445
# Linting
4546
- name: Run flake8

poetry.lock

Lines changed: 25 additions & 25 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pyproject.toml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,14 @@ pytest = "^7.0"
2525
flake8 = "^5.0"
2626
requests-mock = "^1.6"
2727
pytest-cov = "^4.0"
28+
black = "*"
29+
30+
[tool.poetry.group.docs]
31+
optional = true
32+
33+
[tool.poetry.group.docs.dependencies]
2834
mkdocs = "^1.0"
2935
pydoc-markdown = "*"
30-
black = "*"
3136
mkdocs-material = "^6.1.0"
3237

3338
[build-system]

0 commit comments

Comments
 (0)