Skip to content

Improve API diff comment formatting#38100

Open
AndriySvyryd wants to merge 2 commits intomainfrom
ImproveAPIWorkflow
Open

Improve API diff comment formatting#38100
AndriySvyryd wants to merge 2 commits intomainfrom
ImproveAPIWorkflow

Conversation

@AndriySvyryd
Copy link
Copy Markdown
Member

No description provided.

@AndriySvyryd AndriySvyryd requested a review from a team as a code owner April 14, 2026 21:30
Copilot AI review requested due to automatic review settings April 14, 2026 21:30
@AndriySvyryd AndriySvyryd enabled auto-merge (squash) April 14, 2026 21:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adjusts how API baseline diffs are emitted and posted in PR comments to improve readability and reduce visual noise in GitHub.

Changes:

  • Simplifies ApiChief per-type markdown diff output to only include a fenced ```diff block (removing the extra per-type heading).
  • Wraps per-baseline diff output in a collapsible <details> block in the api-review-baselines GitHub workflow.
  • Tweaks workflow wording to refer to “the PR” rather than “the selected PR”.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
eng/Tools/ApiChief/Commands/EmitDelta.cs Removes the per-type ### heading from generated diff markdown, leaving only the fenced diff block.
.github/workflows/api-review-baselines.yml Wraps generated diffs in a <details>/<summary> section and updates related copy.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 14, 2026 21:57
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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.

3 participants