Skip to content

Add publish-pypi workflow#13

Merged
ethanholz merged 2 commits intoomsf:mainfrom
dwhswenson:add-release-script
Aug 28, 2025
Merged

Add publish-pypi workflow#13
ethanholz merged 2 commits intoomsf:mainfrom
dwhswenson:add-release-script

Conversation

@dwhswenson
Copy link
Copy Markdown
Member

Getting ready to do the release! We'll need this workflow in place first

@dwhswenson dwhswenson requested a review from Copilot August 25, 2025 22:14
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

Adds a GitHub Actions workflow to automatically publish the package to PyPI when a new release is published. This prepares the repository for automated package distribution.

  • Configures workflow to trigger on GitHub release publication
  • Sets up Python environment and builds the package using standard tools
  • Uses PyPA's official action for secure PyPI publishing with OIDC

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/workflows/publish-pypi.yaml Outdated
python -m build --sdist --wheel
twine check dist/*
- name: "Publish on pypi"
uses: pypa/gh-action-pypi-publish@release/v1
Copy link

Copilot AI Aug 25, 2025

Choose a reason for hiding this comment

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

Pin the action to a specific commit hash instead of using the mutable 'release/v1' tag to prevent supply chain attacks. For example, use a format like 'pypa/gh-action-pypi-publish@81e9d935c883d0b210363ab89cf05f3894778450'.

Suggested change
uses: pypa/gh-action-pypi-publish@release/v1
uses: pypa/gh-action-pypi-publish@81e9d935c883d0b210363ab89cf05f3894778450

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I understand the recommendation, but since I'm going to do a video on this, I'm going to keep it clear (and recommend that viewers check whatever the most recent hash is).

@dwhswenson dwhswenson requested a review from ethanholz August 25, 2025 22:25
@ethanholz ethanholz merged commit b26afc0 into omsf:main Aug 28, 2025
4 checks passed
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