Skip to content

Remove Mergify configuration - #1347

Merged
jeromekelleher merged 1 commit into
sgkit-dev:mainfrom
Billyzhang1229:ci-drop-mergify
Sep 4, 2026
Merged

Remove Mergify configuration#1347
jeromekelleher merged 1 commit into
sgkit-dev:mainfrom
Billyzhang1229:ci-drop-mergify

Conversation

@Billyzhang1229

@Billyzhang1229 Billyzhang1229 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Closes #1340. Supersedes #1316.

Mergify hasn't merged anything here for a long time: the recent PRs were all merged by hand, and the config is now rejected by Mergify itself (Extra inputs are not permitted @ pull_request_rules → actions → queue → method → rebase), which is why every open PR shows a failing "Mergify Merge Queue" check. bio2zarr and vcztools merge directly on GitHub, so this does the same.

Two things need an admin in the repository settings once this is merged:

  • uninstall the Mergify app, so the failing check disappears;
  • enable "Automatically delete head branches", which replaces the delete_head_branch rule.

@Billyzhang1229 Billyzhang1229 mentioned this pull request Sep 3, 2026
Comment thread .github/dependabot.yml Outdated
- package-ecosystem: github-actions
directory: /
schedule:
interval: monthly

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we leave dependabot out of this for now please? This config will lead to a lot of prs, and it's not clear who will review and merge them

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure, dropped.

Mergify has not been used to merge anything here for a long time: the
recent PRs were all merged by hand, and the config is now rejected by
Mergify itself ("Extra inputs are not permitted ... queue -> method ->
rebase"), which shows up as a failing "Mergify Merge Queue" check on
every PR. bio2zarr and vcztools merge PRs directly on GitHub, so do the
same here.

The app still needs to be uninstalled from the repository settings, and
"Automatically delete head branches" enabled to replace the
delete_head_branch rule.
@Billyzhang1229 Billyzhang1229 changed the title Remove Mergify, add Dependabot for GitHub Actions Remove Mergify configuration Sep 4, 2026
@jeromekelleher
jeromekelleher marked this pull request as ready for review September 4, 2026 12:39
@jeromekelleher
jeromekelleher merged commit 1fc5072 into sgkit-dev:main Sep 4, 2026
12 of 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.

Drop Mergify, move to GitHub approach

2 participants