Skip to content

Conversation

@WilliamBergamin
Copy link
Contributor

Summary

This PR improves the release flow of the project by leveraging OIDC in PyPi

Testing

We need to merge this PR before trying this out against test.pypi.org or running dry runs

Requirements

  • I've read and understood the Contributing Guidelines and have done my best effort to follow them.
  • I've read and agree to the Code of Conduct.
  • I've run ./scripts/install_and_run_tests.sh after making the changes.

@WilliamBergamin WilliamBergamin requested a review from zimeg November 4, 2025 19:33
@WilliamBergamin WilliamBergamin self-assigned this Nov 4, 2025
@WilliamBergamin WilliamBergamin added github_actions Pull requests that update GitHub Actions code release labels Nov 4, 2025
@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.68%. Comparing base (edacafa) to head (24f2021).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   94.68%   94.68%           
=======================================
  Files          11       11           
  Lines         207      207           
=======================================
  Hits          196      196           
  Misses         11       11           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

WilliamBergamin and others added 3 commits November 4, 2025 14:35
…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

@WilliamBergamin These are incredible changes! I think each improvement in the release process makes all changes later better 🙏 ✨

All changes LGTM but I left a few comments that might be polish. Whenever is right, let's test this! 🚀

Comment on lines +10 to +12
- title: 📚 Documentation
labels:
- docs
Copy link
Member

Choose a reason for hiding this comment

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

🔍 note: Not all of our projects have matching labels at the moment... I believe we're moving toward "docs" but we might want to follow up later to make sure projects match here!

Comment on lines 114 to +117
[TestPyPI](https://test.pypi.org/) is a separate instance of the Python Package
Index that allows you to try distribution tools and processes without affecting
the real index. This is useful with changes that relate to the package itself,
example the contents of the `pyproject.toml`
the real index. This is particularly useful when making changes related to the
package configuration itself, for example, modifications to the `pyproject.toml` file.
Copy link
Member

Choose a reason for hiding this comment

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

⭐ praise: Nice! These notes are so useful for the unfamiliar reader!

@WilliamBergamin WilliamBergamin merged commit 57c5e39 into main Nov 4, 2025
17 checks passed
@WilliamBergamin WilliamBergamin deleted the publish-to-test-pypi-with-oidc branch November 4, 2025 20:38
@WilliamBergamin WilliamBergamin mentioned this pull request Nov 4, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code release semver:patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants