Skip to content

Fix output-diff publishing and post a new comment per run#3081

Merged
schani merged 2 commits into
masterfrom
fix/stacked-output-diff-publisher
Jul 23, 2026
Merged

Fix output-diff publishing and post a new comment per run#3081
schani merged 2 commits into
masterfrom
fix/stacked-output-diff-publisher

Conversation

@schani

@schani schani commented Jul 22, 2026

Copy link
Copy Markdown
Member

Changes

Support stacked PRs

The trusted publisher previously required every PR to target master. PR #3080 targets another branch, so its valid artifact was rejected and no report/comment was created.

When workflow_run supplies an exact PR number, validate that number plus the event's head repository, branch, and SHA without requiring a default-branch base. For fork events that omit the PR number, require exactly one open PR associated with the event's head owner and branch.

Post every current comparison as a new comment

Stop finding and patching an older marker-tagged output-diff comment. Every completed comparison for the PR's current head now creates a new comment, whether outputs differ or are clean. Stale comparisons still cannot comment.

Verification

@github-actions

Copy link
Copy Markdown

No generated-output differences

✅ This PR does not change generated outputs.

@schani schani changed the title Publish generated-output reports for stacked PRs Fix output-diff publishing and post a new comment per run Jul 22, 2026
@schani
schani merged commit 5738aed into master Jul 23, 2026
33 of 58 checks passed
@schani
schani deleted the fix/stacked-output-diff-publisher branch July 23, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant