ci: build + attach wheel/sdist to GitHub release (drop PyPI publishing)#21
Merged
Conversation
On a published release, upload the `uv build` artifacts (wheel + sdist) as release assets so consumers can pin a direct `.whl` URL: datamaker-py @ https://github.com/automators-com/datamaker-py/releases/download/<tag>/datamaker_py-<ver>-py3-none-any.whl Because the URL ends in `.whl`, uv/pip install it as a wheel with no build step — the DataMaker desktop runner needs this to provision its venv fully offline (URL-locked sdists currently force a build at install time). The repo is public, so the asset URL needs no auth. Also gate the existing PyPI publish job behind `vars.PUBLISH_TO_PYPI == 'true'`: PyPI trusted publishing isn't set up yet (the project isn't on PyPI), so the job would fail on every release. It's now opt-in until configured. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TMwtcrQZ2WSMZB7wZYM6za
On a published release, build with `uv build` and upload the wheel + sdist as GitHub release assets, so consumers can pin a direct `.whl` URL and install with no build step (the desktop runner provisions its venv fully offline this way). Removes the PyPI trusted-publishing job entirely: the project isn't on PyPI and we're standardizing on GitHub release assets. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TMwtcrQZ2WSMZB7wZYM6za
RatulMaharaj
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The DataMaker desktop runner pins
datamaker-pyby source-archive URL (/archive/<sha>.tar.gz), an sdist that uv/pip must build at install time. That build is what blocks the desktop app from provisioning its Python venv fully offline.What
uv buildand attach the wheel + sdist as GitHub release assets. Consumers pin a direct.whlURL:.whl→ installs as a wheel, no build step. Repo is public → no auth.Next
v0.7.0(wheel attaches automatically)..whlURL + regenerate the lock → runtime does zero building.🤖 Generated with Claude Code