-
Notifications
You must be signed in to change notification settings - Fork 0
feat: prove PCEA 0.1.0 release readiness #36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
10da8a5
19ff70c
e8fc8f4
21d6e66
16fb77b
46c3412
4a93fe7
9976dc3
cf2f8b4
9f7d1d7
c1403e1
2554a32
65cac72
1ddc5f1
f69ca59
4fbb3c5
05f829b
275726c
6b41905
f9668dc
3237ef4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,25 +1,55 @@ | ||
| name: contract-boundary | ||
| name: full-suite | ||
|
|
||
| on: | ||
| pull_request: | ||
| push: | ||
| branches: [ main ] | ||
|
|
||
| jobs: | ||
| boundary: | ||
| test: | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| python-version: ["3.9", "3.11", "3.13"] | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-python@v5 | ||
| - uses: actions/checkout@v7 | ||
| - uses: actions/setup-python@v7 | ||
| with: | ||
| python-version: ${{ matrix.python-version }} | ||
| - name: Install package with dev dependencies | ||
| run: | | ||
| python -m pip install --upgrade pip | ||
| python -m pip install -e ".[dev]" | ||
| - name: Run contract boundary tests | ||
| - name: Run full repository test suite | ||
| run: | | ||
| PYTHONPATH=. python -m pytest -q tests/test_contract_spec.py | ||
| PYTHONPATH=. python -m pytest -q | ||
|
Comment on lines
+23
to
+25
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
After this workflow starts running the entire repository suite, the checked Useful? React with 👍 / 👎. |
||
|
|
||
| release-artifact: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v7 | ||
| - uses: actions/setup-python@v7 | ||
| with: | ||
| python-version: "3.11" | ||
| - name: Build wheel from declared package metadata | ||
| run: | | ||
| python -m pip install --upgrade pip | ||
| python -m pip wheel . --no-deps --wheel-dir dist | ||
| - name: Install wheel outside the source tree and smoke-test public package | ||
| shell: bash | ||
| run: | | ||
| set -euo pipefail | ||
| wheel=$(printf '%s\n' dist/pcea-*.whl) | ||
| test -f "$wheel" | ||
| python -m venv /tmp/pcea-release-smoke | ||
| /tmp/pcea-release-smoke/bin/python -m pip install --no-deps "$wheel" | ||
| cd /tmp | ||
| /tmp/pcea-release-smoke/bin/python - <<'PY' | ||
| from importlib.metadata import version | ||
| import pcea | ||
|
|
||
| assert version("pcea") == "0.1.0" | ||
| assert callable(pcea.encrypt_state) | ||
| assert callable(pcea.decrypt_state) | ||
| assert pcea.PCEAInstance is not None | ||
| PY | ||
|
Comment on lines
+28
to
+55
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On every pull request and push inspected in
.github/workflows/contract-boundary.yml, the job stops while resolving these actions because neither repository publishes av7ref, so the newly expanded pytest suite never runs. The upstream usage examples currently specifyactions/checkout@v6andactions/setup-python@v6; pin these steps to existing releases.Useful? React with 👍 / 👎.