Skip to content

ci: add PyPI publishing workflow and release-please config#10

Merged
khvn26 merged 3 commits into
mainfrom
ci/pypi-publishing
May 20, 2026
Merged

ci: add PyPI publishing workflow and release-please config#10
khvn26 merged 3 commits into
mainfrom
ci/pypi-publishing

Conversation

@khvn26
Copy link
Copy Markdown
Member

@khvn26 khvn26 commented May 19, 2026

Contributes to #6

  • Add release-please config + manifest (Python release type, starting at 0.1.0, bootstrap-sha pinned past the initial commits). The release-please workflow itself is handled by the org GitHub App.
  • Add .github/workflows/publish.yml: tag-triggered, splits build from publish.
    • build job runs make build, attaches dist/* to the GitHub release (gh release upload), and uploads a dist artifact for the publish job.
    • publish job downloads the artifact and publishes to PyPI via OIDC trusted publishing (environment: pypi, id-token: write). No PyPI tokens in the repo.
  • Add make build target wrapping uv build.
  • Bump pyproject.toml to 0.1.0 so it matches the release-please manifest.
  • Add LICENSE

The pypi GitHub environment has already been created on this repo via the API.

Test plan

  • Configure the trusted publisher on PyPI: repo Flagsmith/flagsmith-sql-flag-engine, workflow publish.yml, environment pypi.
  • Merge a release-please PR and confirm tag push triggers publish.yml, dist assets show up on the GitHub release, and the package lands on PyPI.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

File Coverage Missing
All files 100%

Minimum allowed coverage is 100%

Generated by 🐒 cobertura-action against dad913b

Copy link
Copy Markdown

@Zaimwa9 Zaimwa9 left a comment

Choose a reason for hiding this comment

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

One non-blocking NIT

Comment thread .github/workflows/publish.yml
@khvn26 khvn26 merged commit 695717a into main May 20, 2026
3 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.

2 participants