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

PyPI trusted publishing #94

Merged
merged 9 commits into from
Jul 25, 2024
Merged

PyPI trusted publishing #94

merged 9 commits into from
Jul 25, 2024

Conversation

twm
Copy link
Contributor

@twm twm commented Jul 7, 2024

Setup GHA to release to PyPI via trusted publisher.

On PR: Build the package using python -m build and chain it into the tests.

On tag push: build the package and publish to PyPI.

I have created a trusted publisher on PyPI for Incremental. Hopefully this works 🤞 — we'll see when we hit release candidate. I plan to burn through some more tickets first.

@twm twm force-pushed the trusted-publishing branch 4 times, most recently from 8695446 to 24fc0c9 Compare July 7, 2024 22:51
Base automatically changed from 90-pyproject.toml to trunk July 8, 2024 01:03
twm added 2 commits July 7, 2024 18:04
This allows ripgrep to search within it.
@twm twm force-pushed the trusted-publishing branch 2 times, most recently from ec971f5 to bc56fc5 Compare July 8, 2024 01:08
`python -m build` doesn't successfully build with the previous hack
because the setuptools entry points aren't installed. (I am not sure
how it ever worked under tox-wheel — a pre-existing install of
incremental?) So we synthesize the entry points in _build_meta.py.

Is this better than the original hack? 🤷
@twm twm force-pushed the trusted-publishing branch from bc56fc5 to a4d2a7b Compare July 8, 2024 04:27
@twm twm requested a review from a team July 8, 2024 04:39
@twm twm marked this pull request as ready for review July 10, 2024 06:45
_build_meta.py Outdated Show resolved Hide resolved
@twm twm requested review from glyph and a team July 23, 2024 22:17
Copy link
Member

@adiroiban adiroiban left a comment

Choose a reason for hiding this comment

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

Thanks Tom for working on this.

If you are happy with the PR, I think that we can merge it and test it right away by doing a new relese.


After we have it released, I hope we can use hatchling and things will be simplified.

@twm
Copy link
Contributor Author

twm commented Jul 25, 2024

Thank you for the review @adiroiban! I'll issue a release now.

@twm twm merged commit 2bcd106 into trunk Jul 25, 2024
8 checks passed
@twm twm deleted the trusted-publishing branch July 25, 2024 20:05
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