Skip to content

Simplify workflows and enable dual PyPI publishing #88

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
soustruh merged 2 commits into from
Jan 22, 2026
Merged

Simplify workflows and enable dual PyPI publishing #88

soustruh merged 2 commits into from
Jan 22, 2026
+50 −32

Conversation

@soustruh
Copy link
Contributor

@soustruh soustruh commented Jan 22, 2026 •
edited
Loading

Summary

Refactors the deployment workflow to separate concerns and enable proper release publishing.

Changes

  • Docs Generation: Moved to separate manual workflow (generate-docs.yml)

    • Developers run this before creating PRs to update docs
    • No longer auto-generates during deploy (avoids git branch protection issues)

  • PyPI Publishing: Split behavior based on trigger

    • Manual trigger: Publishes to TestPyPI only (for testing)
    • GitHub Release: Publishes to both TestPyPI AND PyPI sequentially

  • Workflow UX: Simplified workflow_dispatch input

    • Replaced environment choice with informational note
    • Clearer expectations: "This workflow publishes to TestPyPI only"

@soustruh soustruh force-pushed the uv branch 2 times, most recently from 0f01817 to 7c3ad37 Compare January 22, 2026 08:03
@soustruh soustruh requested a review from a team January 22, 2026 08:07
kudj
kudj approved these changes Jan 22, 2026
View reviewed changes
Copy link
Contributor

@kudj kudj left a comment

Choose a reason for hiding this comment

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

thanks for changes

@soustruh soustruh merged commit f77d06b into main Jan 22, 2026
2 checks passed
@soustruh soustruh deleted the uv branch January 22, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kudj kudj kudj approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@soustruh @kudj