FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Consolidate CI workflows into one by EpicWink · Pull Request #1265 · opencv/opencv-python · GitHub

Consolidate CI workflows into one - #1265

Open
EpicWink wants to merge 1 commit into
opencv:4.xfrom
EpicWink:single-ci-workflow
Open

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

Conversation

EpicWink commented Aug 24, 2026
edited
Loading

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.

There's likely duplicate between the build and test jobs for each platform, but I didn't want to mess with anything, so it's all just copy-pasted into one file.

The only lost piece of functionality is skipping CI for pull requests if only the other platform's CI configuration has changed. I consider this very minor and unlikely, so I don't think it's worth it to keep.

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

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.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL