Skip to content

Conversation

@mauvilsa
Copy link
Collaborator

No description provided.

@mauvilsa
Copy link
Collaborator Author

@rr- I would like to configure a github action to publish release packages to pypi as observed in this pull request. This not only automates more the release process, but would make sure that the release packages have attestations.

Additional to the changes in this pull request, it is also necessary to configure pypi and the github repo. But I don't have permissions to change settings in either of them. Can you please do so?

What would need to be done is:

  1. Add a Trusted Publisher to the PyPI project, see https://docs.pypi.org/trusted-publishers/adding-a-publisher/
  2. In github create an environment called "pypi" with default settings

@mauvilsa mauvilsa requested a review from rr- July 10, 2025 04:55
@mauvilsa mauvilsa marked this pull request as ready for review July 10, 2025 04:55
Copy link
Owner

@rr- rr- left a comment

Choose a reason for hiding this comment

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

Thank you. I added release.yml and build.yml to the list of permitted integrations on PyPI, and I've added an environment called pypi here on GitHub with default rules. LMK if you need anything more.

@mauvilsa
Copy link
Collaborator Author

After merging this I will try to publish a release candidate to test it out. I don't know how else to test it.

@rr-
Copy link
Owner

rr- commented Jul 10, 2025

Sounds good. We can always go with x.y.0.1 if something goes left, I think.

@mauvilsa mauvilsa merged commit 821aaa5 into master Jul 11, 2025
22 checks passed
@mauvilsa mauvilsa deleted the bumpversion-and-pypi branch July 11, 2025 07:17
@mauvilsa
Copy link
Collaborator Author

Okay, 0.17.0rc1 is released and the workflow was flawless, see https://github.com/rr-/docstring_parser/actions/runs/16214135509. And the attestations are there, see https://pypi.org/project/docstring-parser/0.17.0rc1/#docstring_parser-0.17.0rc1-py3-none-any.whl.

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.

3 participants