Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support merge queue settings for branch protection rules #86

Closed
eclipse-csi-bot opened this issue Jul 27, 2023 · 10 comments
Closed

Support merge queue settings for branch protection rules #86

eclipse-csi-bot opened this issue Jul 27, 2023 · 10 comments
Labels
enhancement New feature or request no-api-available

Comments

@eclipse-csi-bot
Copy link
Contributor

In GitLab by @netomi on Jul 27, 2023, 13:41

@netomi netomi added the enhancement New feature or request label Jan 23, 2024
@mnonnenmacher
Copy link

mnonnenmacher commented Feb 27, 2024

@netomi We will need the merge queue for our repository https://github.com/eclipse-apoapsis/ort-server. As long as this feature is not implemented, can this be manually configured for a repository or would otterdog revert these settings if it runs?

@netomi
Copy link
Contributor

netomi commented Feb 27, 2024

we can manually set the merge queue for now, the remaining settings of a branch protection rule can be setup with otterdog. The unknown setting will not be touched.

@mnonnenmacher
Copy link

Ok, thanks for the confirmation.

@netomi
Copy link
Contributor

netomi commented Feb 27, 2024

so far I encountered nobody using that feature, if there is interest in it I will raise priority accordingly and make sure that is supported asap.

@netomi
Copy link
Contributor

netomi commented Feb 27, 2024

Some research reveals that there is currently no API to create a merge queue, see also this request: https://github.com/orgs/community/discussions/50893

The graphql API allows to query existing merge queue, but there is no mutation to create one.

@netomi
Copy link
Contributor

netomi commented Feb 27, 2024

@mnonnenmacher
Copy link

@netomi https://github.com/orgs/community/discussions/77614 was closed because they can now be configured for rulesets:
https://docs.github.com/en/graphql/overview/changelog#schema-changes-for-2024-07-24

So if Otterdog can support that we might just switch from a branch protection rule to a ruleset, although so far I have not looked into the ruleset feature at all.

@netomi
Copy link
Contributor

netomi commented Sep 25, 2024

ty for the pointer. Rulesets are the same as branch protection rules, just better and more consistent. I expect at some point in time GitHub will deprecate branch protection rules in favor of ruleset.

@netomi
Copy link
Contributor

netomi commented Sep 26, 2024

Created #282 and will merge soon as the implementation is finished.

I wonder if they will add API support for merge queues in branch protection rules as well. In the UI they already point you to rulesets if you want to create a branch protection rule:

image

So as I said before, my guess is they have internally already deprecated them and will not invest any more time into supporting them.

Rulesets are the way to go.

@netomi
Copy link
Contributor

netomi commented Nov 14, 2024

this will probably never be supported by GitHub's API, support for merge queues is not there for rulesets and they are the way to go, so closing for now.

@netomi netomi closed this as not planned Won't fix, can't repro, duplicate, stale Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request no-api-available
Projects
None yet
Development

No branches or pull requests

3 participants