Skip to content

fix: use OIDC trusted publishing for PyPI in release.yml - #69

Merged
farhan merged 1 commit into
mainfrom
restore-oidc-publishing
Aug 20, 2026
Merged

fix: use OIDC trusted publishing for PyPI in release.yml#69
farhan merged 1 commit into
mainfrom
restore-oidc-publishing

Conversation

@farhan

@farhan farhan commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Important

PR implemented with the assistance of Claude Code. Refined and validated before being submitted for code review.

Modernize release.yml
Part of openedx/public-engineering#506

Summary

  • Switch PyPI publishing to OIDC trusted publishing — remove the user: __token__ / password: ${{ secrets.PYPI_UPLOAD_TOKEN }} inputs.

🤖 Generated with Claude Code

Switch the Publish to PyPI step to OIDC trusted publishing, per the Open edX
Python repo modernization effort (openedx/public-engineering#506).

Remove `user: __token__` / `password: ${{ secrets.PYPI_UPLOAD_TOKEN }}`; the
publish_to_pypi job already grants `id-token: write`, so publishing uses OIDC
with no long-lived secret. The PyPI trusted publisher must be configured before
this merges.

Uses a `fix:` type so python-semantic-release cuts a release for this change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.17%. Comparing base (3a9f71e) to head (fbb8e79).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #69   +/-   ##
=======================================
  Coverage   82.17%   82.17%           
=======================================
  Files          48       48           
  Lines        1419     1419           
  Branches      110      110           
=======================================
  Hits         1166     1166           
  Misses        221      221           
  Partials       32       32           
Flag Coverage Δ
unittests 82.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@farhan
farhan merged commit b19a7a5 into main Aug 20, 2026
8 checks passed
@farhan
farhan deleted the restore-oidc-publishing branch August 20, 2026 11:03
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