Skip to content

fix(workflows): normalize report formatting in output templates#21128

Merged
pelikhan merged 2 commits intomainfrom
copilot/normalize-report-formatting-another-one
Mar 15, 2026
Merged

fix(workflows): normalize report formatting in output templates#21128
pelikhan merged 2 commits intomainfrom
copilot/normalize-report-formatting-another-one

Conversation

Copy link
Contributor

Copilot AI commented Mar 15, 2026

Output format templates in several reporting workflows used ##/# headers and lacked <details> progressive disclosure, contradicting the guidelines in shared/reporting.md and causing inconsistent AI-generated output.

Changes

  • weekly-safe-outputs-spec-review.md — Fixed last remaining ## Related Files### Related Files inside the PR body code block template
  • daily-news.md — Wrapped chart embed sections in <details> for progressive disclosure (was the only reporting workflow with zero <details> tags in its output template)
  • weekly-editors-health-check.md — Wrapped editor status table in <details> in the PR body template

All other files listed in the issue had already been updated; these three had residual non-compliance.

Before / After (weekly-editors-health-check.md):

# Before
### Editor Health Report – <date>
| Editor | URL | Status | Preview |
|--------|-----|--------|---------|
...

# After
### Editor Health Report – <date>
<details>
<summary><b>Editor Status & Screenshots</b></summary>

| Editor | URL | Status | Preview |
|--------|-----|--------|---------|
...
</details>

Copilot AI and others added 2 commits March 15, 2026 22:31
…ws (#24204)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review March 15, 2026 22:44
Copilot AI review requested due to automatic review settings March 15, 2026 22:45
@pelikhan pelikhan merged commit 6d9b060 into main Mar 15, 2026
@pelikhan pelikhan deleted the copilot/normalize-report-formatting-another-one branch March 15, 2026 22:45
Copy link
Contributor

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

Normalizes report output templates in a few remaining reporting workflows to match the shared report-formatting guidelines (h3+ headers and progressive disclosure via <details>), reducing inconsistent AI-generated output.

Changes:

  • Adjusts a remaining ## header to ### in the weekly safe-outputs spec review PR body template.
  • Wraps the editor status table in <details> for the weekly editors health check PR body template.
  • Wraps daily news chart embed sections in <details> for progressive disclosure.

Reviewed changes

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

File Description
.github/workflows/weekly-safe-outputs-spec-review.md Normalizes “Related Files” header level to h3 within the PR body template.
.github/workflows/weekly-editors-health-check.md Wraps the editor status table in a collapsible <details> block in the example PR body.
.github/workflows/daily-news.md Wraps chart embed sections in <details> blocks under the Trend Analysis section.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

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.

[workflow-style] Normalize report formatting for non-compliant workflows

3 participants