Skip to content

fix: change tag logic and verbose uv publish#1199

Merged
lucaseduoli merged 1 commit intomainfrom
fix/nightly_tags_pypi
Mar 19, 2026
Merged

fix: change tag logic and verbose uv publish#1199
lucaseduoli merged 1 commit intomainfrom
fix/nightly_tags_pypi

Conversation

@lucaseduoli
Copy link
Collaborator

This pull request makes improvements to the nightly build and publishing process, primarily focusing on the PyPI nightly version tagging logic and the workflow for publishing to PyPI. The changes simplify and clarify the logic for generating nightly build versions, ensuring compliance with PEP 440, and enhance the publishing step for better reliability and debugging.

Nightly versioning logic improvements:

  • Simplified version comparison in pypi_nightly_tag.py to only check the local and main PyPI versions (removing redundant checks for the nightly version), and clarified how the nightly build number is determined, starting at .dev1 and incrementing appropriately if a dev build already exists. [1] [2]

Publishing workflow enhancements:

  • Updated the Publish to PyPI step in .github/workflows/nightly-build.yml to add strict error handling and verbose output for easier debugging and safer publishing.

@lucaseduoli lucaseduoli self-assigned this Mar 19, 2026
@github-actions github-actions bot added ci ⬛ CI/CD, build, and infrastructure issues bug 🔴 Something isn't working. labels Mar 19, 2026
Copy link
Collaborator

@edwinjosechittilappilly edwinjosechittilappilly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot added the lgtm label Mar 19, 2026
@lucaseduoli lucaseduoli merged commit 32420e7 into main Mar 19, 2026
10 checks passed
@github-actions github-actions bot deleted the fix/nightly_tags_pypi branch March 19, 2026 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔴 Something isn't working. ci ⬛ CI/CD, build, and infrastructure issues lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants