Skip to content

fix(ci): run wheel pruning before moving devel tag#334

Merged
pimlock merged 1 commit intomainfrom
fix/ci-prune-devel-wheels-ordering
Mar 16, 2026
Merged

fix(ci): run wheel pruning before moving devel tag#334
pimlock merged 1 commit intomainfrom
fix/ci-prune-devel-wheels-ordering

Conversation

@pimlock
Copy link
Collaborator

@pimlock pimlock commented Mar 16, 2026

Summary

Previous run showing "no release": https://github.com/NVIDIA/OpenShell/actions/runs/23126625930/job/67171875390#step:7:26

Fix the devel wheel pruning step that was silently skipping because it ran after git push --force origin devel.
The force-push disassociates the release from the tag momentarily, causing gh release view devel to 404.

Move the prune step before the tag move so it runs while the old devel release is still intact.

Related Issue

Follow-up to #332 -- the prune step landed but wasn't effective due to step ordering.

Changes

  • Move "Prune stale wheel assets from devel release" step before "Move devel tag" step

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)
  • Parsed .github/workflows/release-dev.yml with Ruby YAML

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

@pimlock pimlock requested a review from drew March 16, 2026 03:54
@pimlock pimlock self-assigned this Mar 16, 2026
@pimlock pimlock merged commit 1b9f35c into main Mar 16, 2026
9 checks passed
@pimlock pimlock deleted the fix/ci-prune-devel-wheels-ordering branch March 16, 2026 07:59
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.

2 participants