Skip to content

Commit f1b413e

Browse files
bhimrazyBorda
andauthored
Use uv for package installations in Makefile docs & test commands (#21167)
* Use `uv` for package installations in Makefile docs cmd to ensure compatibility * apply suggestion --------- Co-authored-by: Jirka Borovec <[email protected]>
1 parent 1fc077b commit f1b413e

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

Makefile

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -45,12 +45,8 @@ clean:
4545
rm -rf src/lightning_fabric/*/
4646
rm -rf src/pytorch_lightning/*/
4747

48-
test: clean
48+
test: clean setup
4949
# Review the CONTRIBUTING documentation for other ways to test.
50-
pip install -e . \
51-
-r requirements/pytorch/base.txt \
52-
-r requirements/fabric/base.txt \
53-
-r requirements/pytorch/test.txt \
5450

5551
# run tests with coverage
5652
python -m coverage run --source src/lightning/pytorch -m pytest src/lightning/pytorch tests/tests_pytorch -v
@@ -59,18 +55,18 @@ test: clean
5955

6056
docs: docs-pytorch
6157

62-
sphinx-theme:
63-
pip install -q awscli
58+
sphinx-theme: setup
59+
uv pip install -q awscli
6460
mkdir -p dist/
6561
aws s3 sync --no-sign-request s3://sphinx-packages/ dist/
66-
pip install lai-sphinx-theme -f dist/
62+
uv pip install lai-sphinx-theme -f dist/
6763

6864
docs-fabric: clean sphinx-theme
69-
pip install -e .[all] --quiet -r requirements/fabric/docs.txt
65+
uv pip install -e '.[all]' --quiet -r requirements/fabric/docs.txt
7066
cd docs/source-fabric && $(MAKE) html --jobs $(nproc)
7167

7268
docs-pytorch: clean sphinx-theme
73-
pip install -e .[all] --quiet -r requirements/pytorch/docs.txt
69+
uv pip install -e '.[all]' --quiet -r requirements/pytorch/docs.txt
7470
cd docs/source-pytorch && $(MAKE) html --jobs $(nproc)
7571

7672
update:

0 commit comments

Comments
 (0)