Skip to content

fix(release): tag the versioned distribution source - #106

Merged
HenryNdubuaku merged 1 commit into
cactus-compute:mainfrom
MohammedThowfiq:codex/tag-versioned-release-source
Sep 2, 2026
Merged

fix(release): tag the versioned distribution source#106
HenryNdubuaku merged 1 commit into
cactus-compute:mainfrom
MohammedThowfiq:codex/tag-versioned-release-source

Conversation

@MohammedThowfiq

Copy link
Copy Markdown
Contributor

Summary

Release tags currently point at the original main commit, while the workflow builds distributions from uncommitted version changes. This makes a source install from the tag report an older version than the published distribution.

Commit the two version metadata files before tagging, keeping that release-only commit on the tag without pushing to main. The daily release check recognizes the tagged commit's source parent so an unchanged main branch still skips publication. Existing tags remain supported.

Fixes #104. This corrects future release tags; existing tags are not rewritten.

Validation

  • python -m pytest tests/test_release.py -q: 2 passed. Runs the workflow shell blocks in temporary Git repositories to verify tagged metadata, old-style tags, reruns on both the release tag and unchanged main, and the next release after a new source commit. The metadata regression fails against the original workflow.
  • python -m pytest -q -m "not slow": 70 passed, 6 skipped, 5 deselected.
  • python -m build: wheel and source distribution built successfully.

The workflow was not run with publishing credentials; tests do not publish packages or push release tags.

Signed-off-by: Mohammedthowfiq <49336054+MohammedThowfiq@users.noreply.github.com>
@MohammedThowfiq
MohammedThowfiq force-pushed the codex/tag-versioned-release-source branch from de3e768 to 34621a0 Compare September 2, 2026 16:19
@HenryNdubuaku
HenryNdubuaku merged commit 205b8f0 into cactus-compute:main Sep 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Source installs report 2.0.8 after v2.0.11

2 participants