Skip to content

Fix emitter diff comments for fork PRs - #5191

Open
Chenjie Shi (tadelesh) wants to merge 1 commit into
Azure:mainfrom
tadelesh:tadelesh/fix-go-emitter-diff-commenter
Open

Fix emitter diff comments for fork PRs#5191
Chenjie Shi (tadelesh) wants to merge 1 commit into
Azure:mainfrom
tadelesh:tadelesh/fix-go-emitter-diff-commenter

Conversation

@tadelesh

Copy link
Copy Markdown
Member

Summary

  • resolve emitter diff commenter PRs using trusted workflow_run head repository and branch data
  • validate the head SHA, head repository, base repository, and unique match before commenting
  • avoid the commit-association lookup that returns no open PR for unmerged fork commits

Validation

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 2422fbee-258a-4c23-a618-df16c85c12df
@github-actions

Copy link
Copy Markdown
Contributor

No changes needing a change description found.

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

You can try these changes here

🛝 Playground 🌐 Website

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants