Skip to content

Add workflow to publish package to PyPi - #2

Merged
kpwtxt merged 2 commits into
mainfrom
add-publish-workflow
Apr 30, 2026
Merged

kpwtxt merged 2 commits into
mainfrom
add-publish-workflow

Conversation

@assignUser

@assignUser assignUser commented Apr 29, 2026 •

Copy link
Copy Markdown
Contributor

This will do a test release to test.pypi.org when run in PRs.

Security wise I have setup the 'pypi' environment to only allow deploys from main and tags of the form 'v*'. This will prevent any accidental pushes to pypi when working on the workflow or similar.

@assignUser

Copy link
Copy Markdown
Contributor Author

The test uploads are here

Comment thread .github/workflows/publish.yml
Comment thread .github/workflows/publish.yml

@kpwtxt kpwtxt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good -- just want to confirm the two comments inline re fetching tags and filtering dev releases

Comment thread .github/workflows/publish.yml Outdated
Comment thread .github/workflows/publish.yml
This will do a test release to test.pypi.org when run in PRs.
We could also do some additional work to create unique versions
etc. but that seems like overkill for this test. The attestation and
trusted publishing will be tested even with this option on.

@kpwtxt kpwtxt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, thanks!

@kpwtxt
kpwtxt merged commit 68fd25f into main Apr 30, 2026
9 checks passed
@kpwtxt

kpwtxt commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

This closes PRO-1526

This branch was successfully deployed

1 active deployment
test-pypi — 4f9d5989 Deployed Apr 29, 2026 by assignUser via Publish to PyPI #9
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.

3 participants