Skip to content

ci: require one summary check instead of every job - #185

Open
dengqiaoyu wants to merge 1 commit into
apple:mainfrom
dengqiaoyu:ci/investigate-simple-ci-for-release
Open

dengqiaoyu wants to merge 1 commit into
apple:mainfrom
dengqiaoyu:ci/investigate-simple-ci-for-release

Conversation

@dengqiaoyu

Copy link
Copy Markdown
Contributor
  • The new All required checks passed job decides whether CI passed: every job must succeed, except the long tests on release-only PRs and macOS outside apple/coreai-optimization.
  • Adding, removing, or renaming test jobs no longer needs a branch rule change, only an edit to that job's needs list.
  • Release-only PRs (only _about.py, CHANGELOG.md, or changelog.d/ changed) skip the tutorial tests and the full macOS and Linux runs.
  • The branch rule must be switched to require All required checks passed after this merges.

This change makes the release-only PRs to quickly pass all check without force merging.

- The new `All required checks passed` job decides whether CI passed: every job must succeed, except the long tests on release-only PRs and macOS outside `apple/coreai-optimization`.
- Adding, removing, or renaming test jobs no longer needs a branch rule change, only an edit to that job's `needs` list.
- Release-only PRs (only `_about.py`, `CHANGELOG.md`, or `changelog.d/` changed) skip the tutorial tests and the full macOS and Linux runs.
- The branch rule must be switched to require `All required checks passed` after this merges.

This branch has not been deployed

No deployments
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