Addressed multiple review comments.

pull/133/head
Robert de Vries 2026-07-06 22:42:35 +02:00
parent 35f9108a5f
commit 0d2e15ffed
3 changed files with 10 additions and 10 deletions

View File

@ -36,8 +36,8 @@ jobs:
run: uv run ruff check run: uv run ruff check
- name: Run tests using the locally built wheel - name: Run tests using the locally built wheel
run: | run: |
uv pip install dist/*.whl uv pip install --reinstall dist/*.whl
uv run pytest tests uv run --no-sync pytest tests
build-no-pqc: build-no-pqc:
# Regression coverage for issue #2659: INVALID_DEVID is only declared # Regression coverage for issue #2659: INVALID_DEVID is only declared
@ -91,8 +91,8 @@ jobs:
uv pip install -e . uv pip install -e .
- name: Check absence of ML-KEM and ML-DSA support - name: Check absence of ML-KEM and ML-DSA support
run: | run: |
uv run python -c "from wolfcrypt._ffi import lib as _lib; assert _lib.ML_KEM_ENABLED == 0, 'ML-KEM should be disabled'" uv run --no-sync python -c "from wolfcrypt._ffi import lib as _lib; assert _lib.ML_KEM_ENABLED == 0, 'ML-KEM should be disabled'"
uv run python -c "from wolfcrypt._ffi import lib as _lib; assert _lib.ML_DSA_ENABLED == 0, 'ML-DSA should be disabled'" uv run --no-sync python -c "from wolfcrypt._ffi import lib as _lib; assert _lib.ML_DSA_ENABLED == 0, 'ML-DSA should be disabled'"
- name: Import smoke (regression for INVALID_DEVID) - name: Import smoke (regression for INVALID_DEVID)
run: uv run python -c "from wolfcrypt.random import Random; Random()" run: uv run python -c "from wolfcrypt.random import Random; Random()"
- name: Run tests - name: Run tests

View File

@ -60,7 +60,7 @@ check: test ## run tests quickly with the default Python
test-all: ## run tests on every Python version with uv test-all: ## run tests on every Python version with uv
for version in 3.10 3.11 3.12 3.13 3.14; do \ for version in 3.10 3.11 3.12 3.13 3.14; do \
echo "=== Python $$version ==="; \ echo "=== Python $$version ==="; \
uv run --python $$version pytest; \ uv run --python $$version pytest tests || exit 1; \
done done
check-all: test-all ## run tests on every Python version with uv check-all: test-all ## run tests on every Python version with uv
@ -71,14 +71,14 @@ cov: ## check code coverage quickly with the default Python
uv run coverage html uv run coverage html
$(BROWSER) htmlcov/index.html $(BROWSER) htmlcov/index.html
docs: install ## generate Sphinx HTML documentation, including API docs docs: ## generate Sphinx HTML documentation, including API docs
$(MAKE) -C docs clean $(MAKE) -C docs clean
$(MAKE) -C docs html uv run $(MAKE) -C docs html
$(BROWSER) docs/_build/html/index.html $(BROWSER) docs/_build/html/index.html
doctest: install ## generate Sphinx HTML documentation, including API docs doctest: ## generate Sphinx HTML documentation, including API docs
$(MAKE) -C docs clean $(MAKE) -C docs clean
$(MAKE) -C docs doctest uv run $(MAKE) -C docs doctest
servedocs: docs ## compile the docs watching for changes servedocs: docs ## compile the docs watching for changes
watchmedo shell-command -p '*.rst' -c '$(MAKE) -C docs html' -R -D . watchmedo shell-command -p '*.rst' -c '$(MAKE) -C docs html' -R -D .

View File

@ -42,7 +42,7 @@ You can then build the source distribution packages using:
Linux Linux
^^^^^ ^^^^^
The `setup.py` file covers most things you will need to do to build and install from source. As pre-requisites you will need to install either from your OS repository or uv. You'll also need the Python development package for your Python version: The `setup.py` file covers most things you will need to do to build and install from source. As pre-requisites you will need to install either from your OS repository or with uv. You'll also need the Python development package for your Python version:
* `uv` * `uv`