Skip to content

ci: fix mergify configuration#1005

Merged
LalatenduMohanty merged 1 commit intopython-wheel-build:mainfrom
tiran:fix-mergify
Apr 1, 2026
Merged

ci: fix mergify configuration#1005
LalatenduMohanty merged 1 commit intopython-wheel-build:mainfrom
tiran:fix-mergify

Conversation

@tiran
Copy link
Copy Markdown
Collaborator

@tiran tiran commented Apr 1, 2026

Pull Request Description

What

PR #903 removed the mypy check and moved all checks to pre-commit. Update mergify configuration to require successful pre-commit, pkglint, and docs builds before a MR can be merged.

@tiran tiran requested a review from a team as a code owner April 1, 2026 11:35
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 05160f9e-5212-4889-871d-8465342f46a5

📥 Commits

Reviewing files that changed from the base of the PR and between 41e04c0 and 2167f28.

📒 Files selected for processing (1)
  • .mergify.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .mergify.yml

📝 Walkthrough

Walkthrough

The .mergify.yml automatic-merge rule was updated: the single check-success=mypy requirement was removed and replaced with three checks: check-success=pre-commit, check-success=pkglint, and check-success=readthedocs (the file comment references check.yaml). All other check-success requirements, including unit and E2E suites, remain unchanged. Lines changed: +4/-2.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: updating the mergify CI configuration to reflect recent check consolidation.
Description check ✅ Passed The description directly relates to the changeset, explaining the context (PR #903 removing mypy) and the specific updates to mergify configuration required checks.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

PR python-wheel-build#903 removed the `mypy` check and moved all checks to `pre-commit`.
Update mergify configuration to require successful `pre-commit`,
`pkglint`, and `docs` builds before a MR can be merged.

Signed-off-by: Christian Heimes <cheimes@redhat.com>
@LalatenduMohanty LalatenduMohanty merged commit 2cf0e7d into python-wheel-build:main Apr 1, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants