Skip to content

ci: Add pypi.yaml to automate publishing to PyPI#54

Merged
ns-rse merged 1 commit intomainfrom
ns-rse/pypi-publishing-workflow
May 29, 2024
Merged

ci: Add pypi.yaml to automate publishing to PyPI#54
ns-rse merged 1 commit intomainfrom
ns-rse/pypi-publishing-workflow

Conversation

@ns-rse
Copy link
Copy Markdown
Collaborator

@ns-rse ns-rse commented May 29, 2024

  • Adds .github/workflows.pypi.yaml to automate publishing of commits tagged with v* to PyPI.
  • Need to build locally and publish manually to setup account first and generate a token for the topostats@shseffield.ac.uk account.

+ Adds `.github/workflows.pypi.yaml` to automate publishing of commits tagged with `v*` to PyPI.
+ Need to build locally and publish manually to setup account first and generate a token for the
  `topostats@shseffield.ac.uk` account.
@ns-rse
Copy link
Copy Markdown
Collaborator Author

ns-rse commented May 29, 2024

  • Need to build locally and publish manually to setup account first and generate a token for the topostats@shseffield.ac.uk account.

Just been to have a look at this and have set up a pending publisher for the AFM-SPM organisation and this repository, which, with a bit of luck may mean that we can tag a release and it will publish to PyPI. 🤞

Copy link
Copy Markdown
Collaborator

@SylviaWhittle SylviaWhittle left a comment

Choose a reason for hiding this comment

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

I haven't done this before myself so I can't verify if the steps will work but I searched online for what each one does and it makes sense.

@ns-rse
Copy link
Copy Markdown
Collaborator Author

ns-rse commented May 29, 2024

It's the same pypy.yaml that is used in TopoStats to publish (I'm lazy!).

I've not tried the Pending Publisher on PyPI before but there is a first time for everything!

@ns-rse ns-rse merged commit 216fef0 into main May 29, 2024
@ns-rse ns-rse deleted the ns-rse/pypi-publishing-workflow branch May 29, 2024 10:54
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