Skip to content

Consolidate release CI workflows into one - #1265

Open
EpicWink wants to merge 2 commits into
opencv:4.xfrom
EpicWink:single-ci-workflow
Open

Consolidate release CI workflows into one#1265
EpicWink wants to merge 2 commits into
opencv:4.xfrom
EpicWink:single-ci-workflow

Conversation

@EpicWink

@EpicWink EpicWink commented Aug 24, 2026

Copy link
Copy Markdown

To reduce duplicate release (aka PyPI publish) jobs. This is primarily to reduce the number of Trusted Publishers required to be configured for #1169.

@EpicWink
EpicWink force-pushed the single-ci-workflow branch from 2593262 to 1a2136c Compare August 24, 2026 10:58
@Mukller

Mukller commented Aug 24, 2026

Copy link
Copy Markdown

This cleanly resolves the biggest operational blocker flagged in #1169 (comment): with publishing consolidated into a single workflow, the trusted-publisher matrix collapses from ~4 workflow filenames × 12 project/index combinations to one filename per project.

Triage note now that three PRs overlap in this area:

Suggest also copying the full PyPI/TestPyPI pending-publisher checklist (all four package names × both indexes) into this PR's description — whoever merges is going to be following it as the setup guide.

@kohtala kohtala mentioned this pull request Aug 25, 2026
@kohtala

kohtala commented Aug 25, 2026

Copy link
Copy Markdown

I commented about this on #1264 (comment): It's enough to have the releases in one workflow for the trusted-publisher matrix. If I read the documentation right the projects are still identified by the environment in the OIDC token (in addition to the workflow file).

For the release, the builds need not be consolidated into the same workflow directly, but can be reused from their workflows.

@EpicWink

Copy link
Copy Markdown
Author

@kohtala you're right, consolidating only the release jobs is sufficient. I've done this

@EpicWink EpicWink changed the title Consolidate CI workflows into one Consolidate release CI workflows into one Aug 26, 2026
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