Skip to content

[AP-2965] update python package index publisher#162

Merged
amofakhar merged 2 commits into
masterfrom
AP-2965_update_pythonuploader
Apr 20, 2026
Merged

[AP-2965] update python package index publisher#162
amofakhar merged 2 commits into
masterfrom
AP-2965_update_pythonuploader

Conversation

@amofakhar
Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings April 20, 2026 12:03
@amofakhar amofakhar requested a review from a team as a code owner April 20, 2026 12:03
@platon-github-app-production
Copy link
Copy Markdown

Comment /request-review to automatically request reviews from the following teams:

You can also request review from a specific team by commenting /request-review team-name, or you can add a description with --notes "<message>"

💡 If you see something that doesn't look right, check the configuration guide.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Changes: Security improvement (1)

This PR updates the PyPI publishing workflow to use the modern OIDC-based publisher action instead of twine + long-lived PyPI credentials.

Changes:

  • Replace setup.py/twine upload publishing with python -m build + pypa/gh-action-pypi-publish.
  • Add id-token: write permissions to support OIDC authentication to PyPI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/pythonpublish.yml Outdated
Comment thread .github/workflows/pythonpublish.yml
Comment thread .github/workflows/pythonpublish.yml Outdated
@amofakhar amofakhar merged commit 0a5f68e into master Apr 20, 2026
8 checks passed
@amofakhar amofakhar deleted the AP-2965_update_pythonuploader branch April 20, 2026 12:40
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