Skip to content

Commit fe10b68

Browse files
committed
chore(python): remove the SDK wheel verifier and build task
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
1 parent c16f57f commit fe10b68

6 files changed

Lines changed: 3 additions & 346 deletions

File tree

‎.github/workflows/branch-checks.yml‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -666,8 +666,7 @@ jobs:
666666
set -euo pipefail
667667
changed=$(git diff --name-only --diff-filter=d "${BASE_SHA}...HEAD" -- \
668668
python \
669-
tasks/scripts/generate_python_proto.py \
670-
tasks/scripts/verify-python-wheel.py)
669+
tasks/scripts/generate_python_proto.py)
671670
if [ -z "$changed" ]; then
672671
echo "No added or changed Python SDK files."
673672
exit 0

‎AGENTS.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ Follow `CONTRIBUTING.md` and `create-github-pr`. Authors without target-reposito
200200
## Python SDK (`python/openshell/`)
201201

202202
- The Python SDK moved to [ai-openshell/sdk-python](https://github.com/ai-openshell/sdk-python), which publishes `openshell` to PyPI. Open SDK changes there.
203-
- `python/` and the SDK build scripts `tasks/scripts/generate_python_proto.py` and `tasks/scripts/verify-python-wheel.py` are frozen until core removes them; Branch Checks fails PRs that change them. Put tests for core Python scripts next to the scripts, not under `python/`.
203+
- `python/` and the SDK build script `tasks/scripts/generate_python_proto.py` are frozen until core removes them; Branch Checks fails PRs that change them. Put tests for core Python scripts next to the scripts, not under `python/`.
204204

205205
## TypeScript SDK (`sdk/typescript/`)
206206

‎python/wheel_verifier_test.py‎

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

‎tasks/ci.toml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
[build]
77
description = "Build the whole project"
8-
depends = ["build:rust:workspace", "build:docker", "build:python:wheel"]
8+
depends = ["build:rust:workspace", "build:docker"]
99

1010
["build:rust"]
1111
description = "Alias for build:rust:workspace"

‎tasks/python.toml‎

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -10,32 +10,6 @@ env = { SETUPTOOLS_SCM_PRETEND_VERSION_FOR_OPENSHELL = "0.0.0" }
1010
run = "uv sync --group dev"
1111
hide = true
1212

13-
["build:python:wheel"]
14-
description = "Build the pure-Python SDK wheel"
15-
depends = ["python:proto"]
16-
run = """
17-
#!/usr/bin/env bash
18-
set -euo pipefail
19-
20-
WHEEL_OUTPUT_DIR=${WHEEL_OUTPUT_DIR:-target/wheels}
21-
rm -rf "$WHEEL_OUTPUT_DIR"
22-
mkdir -p "$WHEEL_OUTPUT_DIR"
23-
24-
uv build --wheel --out-dir "$WHEEL_OUTPUT_DIR"
25-
26-
VERIFY_ARGS=()
27-
if [ -n "${SETUPTOOLS_SCM_PRETEND_VERSION_FOR_OPENSHELL:-}" ]; then
28-
VERIFY_ARGS+=(--expected-version "$SETUPTOOLS_SCM_PRETEND_VERSION_FOR_OPENSHELL")
29-
fi
30-
uv run --no-project python tasks/scripts/verify-python-wheel.py "$WHEEL_OUTPUT_DIR" ${VERIFY_ARGS[@]+"${VERIFY_ARGS[@]}"}
31-
"""
32-
hide = true
33-
34-
["python:build"]
35-
description = "Alias for build:python:wheel"
36-
depends = ["build:python:wheel"]
37-
hide = true
38-
3913
["python:lint"]
4014
description = "Lint Python code with ruff"
4115
depends = ["python:proto"]

‎tasks/scripts/verify-python-wheel.py‎

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

0 commit comments

Comments
 (0)