Skip to content

Upload package upon release#184

Merged
gmloose merged 15 commits intomasterfrom
RAP-994_upload-package-upon-release
Oct 20, 2025
Merged

Upload package upon release#184
gmloose merged 15 commits intomasterfrom
RAP-994_upload-package-upon-release

Conversation

@gmloose
Copy link
Collaborator

@gmloose gmloose commented Oct 20, 2025

Automatically upload the Python package (source and wheel) to PyPI when a release (tag) is created.

@gmloose gmloose requested a review from tammojan October 20, 2025 13:51
@gmloose gmloose self-assigned this Oct 20, 2025
@github-actions
Copy link

github-actions bot commented Oct 20, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
4361 765 18% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 1a65ca1 by action🐍

@gmloose gmloose marked this pull request as draft October 20, 2025 13:57
Copy link
Collaborator

@tammojan tammojan left a comment

Choose a reason for hiding this comment

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

Looks good to me. These things are best tested on actually making a release, so if there's a small error that I'm overlooking feel free to submit that in a follow-up. Please do squash commits when merging, there are quite a few work-in-progress commits on this branch.

@gmloose gmloose marked this pull request as ready for review October 20, 2025 15:18
@gmloose gmloose requested a review from tammojan October 20, 2025 15:18
@gmloose
Copy link
Collaborator Author

gmloose commented Oct 20, 2025

Looks good to me. These things are best tested on actually making a release, so if there's a small error that I'm overlooking feel free to submit that in a follow-up. Please do squash commits when merging, there are quite a few work-in-progress commits on this branch.

I made a couple of changes, especially about when this workflow should run. And I tested it by uploading to PyPI. Would you mind reviewing again?

Copy link
Collaborator

@tammojan tammojan left a comment

Choose a reason for hiding this comment

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

Looks good to me again :)

@gmloose gmloose merged commit 3335b05 into master Oct 20, 2025
13 of 14 checks 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.

2 participants