Skip to content

Split publish workflows into two jobs#85

Merged
kevinbackhouse merged 1 commit into
GitHubSecurityLab:mainfrom
kevinbackhouse:split-publish-workflows
Jun 3, 2026
Merged

Split publish workflows into two jobs#85
kevinbackhouse merged 1 commit into
GitHubSecurityLab:mainfrom
kevinbackhouse:split-publish-workflows

Conversation

@kevinbackhouse
Copy link
Copy Markdown
Contributor

Apply the same changes as what we already did in seclab-taskflow-agent (see GitHubSecurityLab/seclab-taskflow-agent#246).

Copilot AI review requested due to automatic review settings June 2, 2026 14:56
anticomputer
anticomputer previously approved these changes Jun 2, 2026
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

This PR updates the release automation to mirror the approach used in seclab-taskflow-agent: separating build and publish concerns so publication runs only after a completed build and uses downloaded artifacts.

Changes:

  • Split both PyPI/TestPyPI publish workflows into build (build + upload artifacts) and publish (download + attest + publish + GitHub release) jobs.
  • Reduce build-job permissions to contents: read and move publish-only permissions/environment to the publish job.
  • Update several GitHub Actions versions and bump the build Python version to 3.14.
Show a summary per file
File Description
.github/workflows/publish-to-testpypi.yaml Splits TestPyPI pre-release publishing into build/publish jobs with artifact handoff and provenance attestation.
.github/workflows/publish-to-pypi.yaml Splits PyPI publishing into build/publish jobs with artifact handoff and provenance attestation.

Copilot's findings

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 4

Comment on lines +78 to +81
- name: Attest build provenance
uses: actions/attest@v4.1.0 # immutable release
with:
subject-path: ./dist/*
Comment on lines 21 to 25
- name: Set up Python
uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
with:
python-version: "3.13"
python-version: "3.14"

Comment on lines +65 to +68
- name: Attest build provenance
uses: actions/attest@v4.1.0 # immutable release
with:
subject-path: ./dist/*
Comment on lines 23 to 27
- name: Set up Python
uses: actions/setup-python@e797f83bcb11b83ae66e0230d6156d7c80228e7c # v6.0.0
uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
with:
python-version: "3.13"
python-version: "3.14"

@kevinbackhouse kevinbackhouse merged commit f10c99a into GitHubSecurityLab:main Jun 3, 2026
6 checks passed
@kevinbackhouse kevinbackhouse deleted the split-publish-workflows branch June 3, 2026 15:44
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