Automate PyPI Publishing via GitHub Actions#31
Automate PyPI Publishing via GitHub Actions#31yunks128 wants to merge 23 commits intoNASA-AMMOS:mainfrom
Conversation
riverma
left a comment
There was a problem hiding this comment.
@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.
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