Skip to content

Adds PyPi and fix SBOM generation#124

Merged
vpetersson merged 2 commits intomasterfrom
pipy
Jan 18, 2026
Merged

Adds PyPi and fix SBOM generation#124
vpetersson merged 2 commits intomasterfrom
pipy

Conversation

@vpetersson
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 18, 2026 15:52
Copy link
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

This PR adds PyPI publishing workflows and refactors SBOM generation jobs to improve clarity and correctness. The changes separate staging and production SBOM generation into distinct jobs with proper conditional execution, eliminating complex runtime checks.

Changes:

  • Added a new PyPI/TestPyPI publishing workflow triggered on releases
  • Split SBOM generation jobs into separate staging and production workflows with job-level conditionals
  • Removed runtime should_run checks in favor of declarative if conditions at the job level

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/sbomify.yaml Refactored SBOM generation by splitting into separate staging/production jobs with clearer conditionals
.github/workflows/pypi.yaml Added new workflow for publishing Python packages to PyPI and TestPyPI on releases

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

echo "$HOME/.local/bin" >> $GITHUB_PATH

- name: Build a binary wheel and a source tarball
run: |
Copy link

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

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

The --frozen flag requires an exact lock file match, which may cause build failures if the lock file is out of sync. Consider using uv sync without --frozen or adding a verification step to ensure the lock file is up to date before building.

Suggested change
run: |
run: |
uv lock

Copilot uses AI. Check for mistakes.
@vpetersson vpetersson merged commit 2ca0ab5 into master Jan 18, 2026
11 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.

1 participant