Skip to content

Fix update-pull-request generated footer spacing - #55372

Merged
pelikhan merged 1 commit into
mainfrom
copilot/update-footer-insertion-formatting
Aug 24, 2026
Merged

Fix update-pull-request generated footer spacing#55372
pelikhan merged 1 commit into
mainfrom
copilot/update-footer-insertion-formatting

Conversation

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

update_pull_request could append the generated footer directly after new body content, causing malformed PR descriptions such as run: ...> Generated by .... This adds an explicit blank-line boundary before the generated footer across PR body update modes.

  • Body assembly

    • Adds a shared footer section that prefixes generated attribution with \n\n.
    • Applies the same spacing to replace, append, prepend, and replace-island.
  • Regression coverage

    • Adds focused cases that verify generated footers stay separated from body content.
    • Covers the steering-style run: ... body shape that exposed the formatting issue.

Example output shape:

run: https://github.com/owner/repo/actions/runs/123

> Generated by [Workflow](https://github.com/owner/repo/actions/runs/123)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Fix update-pull-request footer spacing Fix update-pull-request generated footer spacing Aug 24, 2026
Copilot AI requested a review from pelikhan August 24, 2026 11:53
@pelikhan
pelikhan marked this pull request as ready for review August 24, 2026 11:56
Copilot AI balanced review requested due to automatic review settings August 24, 2026 11:56
@pelikhan
pelikhan merged commit 0cb6fb2 into main Aug 24, 2026
20 of 24 checks passed
@pelikhan
pelikhan deleted the copilot/update-footer-insertion-formatting branch August 24, 2026 12:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Ensures generated attribution footers remain separated from updated PR body content.

Changes:

  • Adds a shared blank-line-prefixed footer section across all update modes.
  • Adds regression tests for replace, append, prepend, and replace-island.
Show a summary per file
File Description
actions/setup/js/update_pr_description_helpers.cjs Adds consistent footer spacing during body assembly.
actions/setup/js/update_pr_description_helpers.test.cjs Verifies footer separation for every update mode.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.5

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