Skip to content
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

Adding Github action to automatically publish to pypi #271

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

davereinhart
Copy link
Contributor

Publishing id3c to pypi will help standardize how this package is being
installed compared to other dependencies. This will make the package simpler
include in multiple applications and pin to a specific version when needed.

One workflow change this will require is that commits on master branch will need to
be tagged to be published to pypi.org (whereas all commits on master branch will be
published to test.pypi.org). A version naming convention will need to be
decided on and integrated into the deployment workflow, but the specific format
chosen will not impact this PR.

Publishing id3c to pypi will help standardize how this package is being
installed compared to other dependencies. This will make the package simpler
include in multiple applications and pin to a specific version when needed.

One workflow change this will require is that commits on master branch will need to
be tagged to be published to pypi.org (whereas all commits on master branch will be
published to test.pypi.org). A version naming convention will need to be
decided on and integrated into the deployment workflow, but the specific format
chosen will not impact this PR.
@davereinhart davereinhart requested a review from a team as a code owner November 29, 2021 20:41
@davereinhart
Copy link
Contributor Author

A simple version naming convention was already established, so one option is to stick with that:
538ba22

Publishing to test.pypi.org on each push to master creates a couple of
challenges. First pypi cannot overwrite a file with the same name,
which is the case when publishing a package that does not have a different
reference tag in github. There are some solutions to work around this by
having a specific "RC" tag format for release candidates and only publishing
those to test.pypi.org. At this point that would create additional unnecessary
steps in the deployment process but may be worth revisiting in the future once
there is a versioning/tagging conventions are established.
@davereinhart
Copy link
Contributor Author

Note to self: this needs to be rebased and updated to use Python 3.9

@davereinhart davereinhart marked this pull request as draft December 2, 2022 16:38
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