Skip to content

mergify: disable temporary PR branches#117

Merged
garlick merged 1 commit intoflux-framework:mainfrom
grondo:mergify-noparallel
Sep 30, 2025
Merged

mergify: disable temporary PR branches#117
garlick merged 1 commit intoflux-framework:mainfrom
grondo:mergify-noparallel

Conversation

@grondo
Copy link
Copy Markdown
Contributor

@grondo grondo commented Sep 30, 2025

Problem: Mergify is creating temporary PR branches and merging the main branch (plus possibly other PRs). The merge commit causes the pr-validator to fail, plus in the case of an up-to-date original PR, the checks are redundant.

This is occurring because the maximum parallel checks default was updated to 5 by mergify:

https://changelog.mergify.com/changelog/maxparallelchecks-is-now-5-by-default

Explicitly set max_parallel_checks to 1 to avoid the temporary branches. Also set batch_size to 1 in queue_rules for good measure, since we currently never want to batch up PRs for merging.

Problem: Mergify is creating temporary PR branches and merging the
main branch (plus possibly other PRs). The merge commit causes the
pr-validator to fail, plus in the case of an up-to-date original PR,
the checks are redundant.

This is occurring because the maximum parallel checks default was
updated to 5 by mergify:

https://changelog.mergify.com/changelog/maxparallelchecks-is-now-5-by-default

Explicitly set max_parallel_checks to 1 to avoid the temporary
branches. Also set batch_size to 1 in queue_rules for good measure,
since we currently never want to batch up PRs for merging.
Copy link
Copy Markdown
Member

@garlick garlick left a comment

Choose a reason for hiding this comment

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

Thanks for running this down!

@grondo
Copy link
Copy Markdown
Contributor Author

grondo commented Sep 30, 2025

I think this will need a manual merge due to the modification of the mergify config

@garlick garlick merged commit 4bfccfc into flux-framework:main Sep 30, 2025
13 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