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

Add support for merge queues #1061

Merged
merged 1 commit into from
Mar 22, 2023
Merged

Add support for merge queues #1061

merged 1 commit into from
Mar 22, 2023

Conversation

dmikusa
Copy link
Contributor

@dmikusa dmikusa commented Mar 18, 2023

Summary

This PR adds a merge group event type which can be added to workflows as a trigger. It exposes the documented types, of which there is only one, and it adds filters for branch and path (these are not documented, but I see references to them in other projects and they are needed to make this useful). It also updates the tests workflow so that tests are run when a merge group event is detected. It does not add label checks or action workflows. Those will run on PR, which is probably sufficient. We just want to re-run the tests in the merge queue to validate.

This PR has not been tested because we cannot use the merge queue feature yet. EasyCLA is not compatible. This is an attempt to prepare but a follow up PR may be required. Consider this experimental for now.

Resolves #1036

This PR adds a merge group event type which can be added to workflows as a trigger. It exposes the documented types, of which there is only one, and it adds filters for branch and path (these are not documented, but I see references to them in other projects and they are needed to make this useful). It also updates the tests workflow so that tests are run when a merge group event is detected. It does not add label checks or action workflows. Those will run on PR, which is probably sufficient. We just want to re-run the tests in the merge queue to validate.

This PR has not been tested because we cannot use the merge queue feature yet. EasyCLA is not compatible. This is an attempt to prepare but a follow up PR may be required. Consider this experimental for now.

Resolves #1036

Signed-off-by: Daniel Mikusa <dan@mikusa.com>
@dmikusa dmikusa added type:enhancement A general enhancement semver:minor A change requiring a minor version bump labels Mar 18, 2023
@dmikusa dmikusa requested a review from a team as a code owner March 18, 2023 03:56
@dmikusa dmikusa merged commit 17f0456 into main Mar 22, 2023
@dmikusa dmikusa deleted the gh_1036 branch March 22, 2023 02:39
This was referenced Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:minor A change requiring a minor version bump type:enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for merge queues
1 participant