Skip to content

RFC: fix CI, add PyPI publish action, poetry 1.6.1#9

Merged
MartinCura merged 20 commits intomasterfrom
add-pypi-publish-action
Oct 19, 2023
Merged

RFC: fix CI, add PyPI publish action, poetry 1.6.1#9
MartinCura merged 20 commits intomasterfrom
add-pypi-publish-action

Conversation

@leo-mansini
Copy link
Copy Markdown
Contributor

Adds github action to publish to pypi. Also tries with testpypi on every push (not just a tag release).

@leo-mansini leo-mansini changed the title RFC: Add pypi publish action WIP: Add pypi publish action Oct 18, 2023
Comment thread .github/workflows/publish.yml Outdated
Comment thread .github/workflows/publish.yml Outdated

jobs:
pypi-publish:
if: startsWith(github.ref, 'refs/tags/')
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.

this makes sure that this runs only when a tag is pushed, right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, I couldn't find a more robust way to do it, but it's the suggested way in the action documentation and in the python guide.

Copy link
Copy Markdown
Contributor

@MartinCura MartinCura Oct 19, 2023

Choose a reason for hiding this comment

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

great, adjusted it a bit based on the action's recommendations
image

Co-authored-by: martín <MartinCura@users.noreply.github.com>
...maybe missing API token for prod publish if not doing Trusted
Publishing
@MartinCura MartinCura temporarily deployed to testpypi October 19, 2023 00:12 — with GitHub Actions Inactive
@MartinCura MartinCura temporarily deployed to testpypi October 19, 2023 14:10 — with GitHub Actions Inactive
@MartinCura MartinCura changed the title WIP: Add pypi publish action RFC: Add PyPI publish action Oct 19, 2023
Copy link
Copy Markdown
Contributor Author

@leo-mansini leo-mansini left a comment

Choose a reason for hiding this comment

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

@MartinCura Can't approve as I opened the PR but consider it approved. 📦 🐍

Comment thread .github/workflows/publish.yml Outdated
@MartinCura MartinCura temporarily deployed to testpypi October 19, 2023 14:31 — with GitHub Actions Inactive
@MartinCura MartinCura temporarily deployed to testpypi October 19, 2023 14:41 — with GitHub Actions Inactive
@MartinCura MartinCura temporarily deployed to testpypi October 19, 2023 14:57 — with GitHub Actions Inactive
@MartinCura MartinCura temporarily deployed to testpypi October 19, 2023 15:37 — with GitHub Actions Inactive
@MartinCura MartinCura changed the title RFC: Add PyPI publish action RFC: fix CI, add PyPI publish action, poetry 1.6.1 Oct 19, 2023
@MartinCura MartinCura merged commit e5d5b86 into master Oct 19, 2023
@MartinCura MartinCura deleted the add-pypi-publish-action branch October 19, 2023 16:01
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