-
Notifications
You must be signed in to change notification settings - Fork 3
Automate PyPI Publishing via GitHub Actions #31
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.
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
base: main
Are you sure you want to change the base?
Conversation
riverma
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yunks128 - would you be able to lookup best practices on PyPI / GitHub workflow automation? It might be a good idea to monitor main for changes and automatically push to PyPI whenever main changes (as opposed to tagged branches). The alternative is tagged branches as you've suggested, but it'd be good to ensure we don't need to update the version in any code file like the workflow file.
|
Python 3.7 was end-of-life in June of '23. Python 3.8 was EOL in October '24.1 I would respectfully suggest modifying Footnotes
|
|
Also, this example is a little different but it is using Trusted Publishing which ensures that releases are marked as trusted in the PyPi UI (otherwise they report unverified origin). Note that there is no explicit declaration of API tokens. It's also different because the project layout is using the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR automates the process of publishing the SLIM CLI Python package to PyPI by introducing a new GitHub Actions workflow and updating the README with detailed publishing instructions.
- Added a GitHub Actions workflow that extracts the version from pushed tags, updates VERSION.txt, builds the package, and publishes it to PyPI.
- Updated the README to include step-by-step instructions for creating and pushing version tags, as well as how to remove tags if necessary.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| README.md | Updated release instructions with detailed steps |
| .github/workflows/publish-to-pypi.yml | New workflow to automate version extraction, build, and PyPI publishing |
This PR is implementing a different approach to release management. Before re-reviewing the topic should be discussed further to provide a more topical and meaningful review.
|
@riverma: consider google's "release please" |
This reverts commit b58da21.
|



Purpose
Proposed Changes
.github/workflows/publish-to-pypi.ymlthat:src/jpl/slim/VERSION.txtbuildtwineREADME.mdwith detailed instructions on how to tag a new release and trigger publishing workflowIssues
Testing
v0.0.8VERSION.txtupdate and successful build stepPYPI_API_TOKENsecret