Skip to content

Commit 1f6336c

Browse files
authored
sdk/docs: Move documentation dependencies to pyproject.toml (#389)
Previously, the build-dependencies for the sphinx autodocumentation were listed in `sdk/docs/add-requirements.txt`. This is outdated and the new state of the art is managing all dependencies via `pyproject.toml`. This refactors the dependencies from `sdk/docs/add-requirements.txt` into a new optional `[docs]` section in the `pyproject.toml` dependency definitions. Furthermore, we adapt the `.github/workflows/ci.yml` to use the new definitions accordingly. Fixes #384
1 parent f780c47 commit 1f6336c

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,7 @@ jobs:
161161
- name: Install Python dependencies
162162
run: |
163163
python -m pip install --upgrade pip
164-
pip install .
165-
pip install -r docs/add-requirements.txt
164+
pip install .[docs]
166165
- name: Check documentation for errors
167166
run: |
168167
SPHINXOPTS="-a -E -n -W --keep-going" make -C docs html

sdk/.readthedocs.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,5 @@ python:
1515
install:
1616
- method: pip
1717
path: .
18-
- requirements: docs/add-requirements.txt
18+
extra_requirements:
19+
- docs

sdk/docs/add-requirements.txt

Lines changed: 0 additions & 4 deletions
This file was deleted.

sdk/pyproject.toml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,11 @@ dev = [
5555
"types-python-dateutil",
5656
"lxml-stubs~=0.5.1",
5757
]
58+
docs= [
59+
"sphinx~=8.2",
60+
"sphinx-rtd-theme~=3.0",
61+
"sphinx-argparse~=0.5.0"
62+
]
5863

5964
[project.urls]
6065
"Homepage" = "https://github.com/eclipse-basyx/basyx-python-sdk"

0 commit comments

Comments
 (0)