Skip to content

Report Generation#837

Merged
jeo02 merged 7 commits intomicrosoft:mainfrom
jeo02:report-generation
Feb 10, 2026
Merged

Report Generation#837
jeo02 merged 7 commits intomicrosoft:mainfrom
jeo02:report-generation

Conversation

@jeo02
Copy link
Collaborator

@jeo02 jeo02 commented Feb 9, 2026

  • If there are multiple markdown files generated in one directory (likely a skill directory) make a single report.
  • Generate a "master" report that summarizes the whole run.

Copilot AI review requested due to automatic review settings February 9, 2026 23:48
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

Updates the test report generation script to consolidate multiple per-test markdown artifacts into one report per test subdirectory, and optionally generate a master report summarizing the entire test run.

Changes:

  • Generate a single consolidated report per subdirectory (instead of per markdown file).
  • Add a master report that aggregates all subdirectory reports when multiple exist.
  • Introduce naming constants and template existence validation.

Copilot AI review requested due to automatic review settings February 9, 2026 23:52
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

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

Comments suppressed due to low confidence (1)

tests/scripts/generate-test-reports.ts:13

  • The usage examples mention npm run reports, but tests/package.json defines the script as npm run report. Update the header comment so the documented command matches the actual npm script name.
 * Usage:
 *   npm run reports              # Process most recent test run
 *   npm run reports <path>       # Process specific test run
 */

JasonYeMSFT
JasonYeMSFT previously approved these changes Feb 9, 2026
Copilot AI review requested due to automatic review settings February 10, 2026 00:04
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

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

@jeo02 jeo02 merged commit cff8624 into microsoft:main Feb 10, 2026
15 checks passed
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