Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix for merge_group event bug #846

Merged
merged 5 commits into from
Oct 28, 2024
Merged

Fix for merge_group event bug #846

merged 5 commits into from
Oct 28, 2024

Conversation

Ahmed3lmallah
Copy link
Contributor

@Ahmed3lmallah Ahmed3lmallah commented Oct 28, 2024

Closes: #841

We incorrectly treated the merge_group and pull_request events as if they had the same schema.

Here is the schema for the merge_group event: https://docs.github.com/en/webhooks/webhook-events-and-payloads#merge_group

This PR adds support for the merge_group event in the getRefs function.

Example run: https://github.com/dsp-testing/ae-example-javascript/actions/runs/11548421420/job/32139809147 [for internal reviewers]

@Ahmed3lmallah Ahmed3lmallah requested a review from a team as a code owner October 28, 2024 06:20
@Ahmed3lmallah Ahmed3lmallah merged commit 4081bf9 into main Oct 28, 2024
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Action fails in merge queue with v4.3.5
2 participants