Skip to content

feat: added fallback for finding pr number #420

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Nov 26, 2025
Merged

feat: added fallback for finding pr number #420

merged 6 commits into from
Nov 26, 2025

Conversation

@Ganga4060
Copy link
Contributor

@Ganga4060 Ganga4060 commented Nov 13, 2025

Improve PR Number Detection in workflow
This pull request updates the workflow to detect the PR number .

What’s changed
Adds a fallback mechanism:

  • If the PR number is not found in the event payload, the workflow now searches all open PRs for one whose head commit SHA matches the check suite’s SHA.

Why

  • Previously, the workflow could not detect the PR number for PRs from forks or in some upstream scenarios, causing log export and PR commenting to be skipped.

@Ganga4060 Ganga4060 requested a review from a team as a code owner November 13, 2025 07:39
@Ganga4060 Ganga4060 requested a review from anubhav756 November 13, 2025 08:37
anubhav756
anubhav756 previously approved these changes Nov 17, 2025
View reviewed changes
@Ganga4060 Ganga4060 requested a review from anubhav756 November 24, 2025 11:58
@Ganga4060 Ganga4060 merged commit eb73f1d into googleapis:main Nov 26, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anubhav756 anubhav756 anubhav756 approved these changes

Assignees

@Ganga4060 Ganga4060

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Ganga4060 @anubhav756 @dishaprakash