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

[EngSys] Extract PR number from check_suite:completed event from fork PR (if possible) #32547

Closed
mikeharder opened this issue Feb 10, 2025 · 1 comment
Assignees

Comments

@mikeharder
Copy link
Member

mikeharder commented Feb 10, 2025

In a fork PR, the pull_requests property in the payload will be empty for fork PRs. For workflow_run:completed, it's possible to extract the PR number from the payload, by calling an API in the head (fork) repo.

So far, I haven't figured out how to do the same for check_suite:completed on fork PRs. This is needed for checks like "ARM Auto Signoff (Preview)".

The check_suite:completed event provides the head_sha, but not the owner of the fork repo containing the SHA.

@mikeharder mikeharder self-assigned this Feb 10, 2025
@mikeharder mikeharder moved this from 🤔 Triage to 🐝 Dev in Azure SDK EngSys 🤖🧠 Feb 10, 2025
@mikeharder
Copy link
Member Author

Appears to be impossible with the information in check_suite:completed from fork PRs. Workaround was to trigger on issue_comment:edited if body contains "next steps to merge" (#32554)

@mikeharder mikeharder closed this as not planned Won't fix, can't repro, duplicate, stale Feb 11, 2025
@github-project-automation github-project-automation bot moved this from 🐝 Dev to 🎊 Closed in Azure SDK EngSys 🤖🧠 Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

1 participant