Skip to content

Conversation

@cmoussa1
Copy link
Member

@cmoussa1 cmoussa1 commented Oct 2, 2025

Problem

Mergify is complaining that the setting to require branches to be up to date before merging is not compatible with draft PR checks.


This PR updates the Mergify configuration to enable in-place checks by setting merge_queue.max_parallel_checks to 1 and setting every queue_rule's batch_size to 1.

I noticed that status checks are not added or configured for this repo, so perhaps we should set that up before merging this PR since it removes the status-success checks in the conditions section.

Because this is making a change to mergify config, this will need a manual merge.

Problem: Mergify is complaining that the setting to require branches to
be up to date before merging is not compatible with draft PR checks.

Update the Mergify configuration to enable in-place checks by setting
merge_queue.max_parallel_checks to 1 and setting every queue_rule's
batch_size to 1.
Copy link
Contributor

@grondo grondo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I've added branch protections for the status checks and will merge shortly.

@grondo grondo merged commit 32b68e5 into flux-framework:master Oct 2, 2025
6 checks passed
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