Skip to content

Split AzureDevOpsSummaryReporter into focused modules - #10644

Open
Amaury Levé (Evangelink) wants to merge 1 commit into
mainfrom
dev/amauryleve/split-azure-devops-summary-reporter
Open

Split AzureDevOpsSummaryReporter into focused modules#10644
Amaury Levé (Evangelink) wants to merge 1 commit into
mainfrom
dev/amauryleve/split-azure-devops-summary-reporter

Conversation

@Evangelink

Copy link
Copy Markdown
Member

Summary

  • split AzureDevOpsSummaryReporter into focused core, lifecycle/path, and Markdown partial modules
  • preserve lifecycle ordering, cancellation/error handling, path resolution, test hooks, and Markdown rendering exactly
  • keep every partial file below 300 lines with UTF-8 BOM

Validation

  • Microsoft.Testing.Extensions.UnitTests passed on net462, net472, net8.0, and net9.0
  • full repository pack succeeded
  • all 9 HelpInfoAllExtensionsTests acceptance executions passed on net10.0
  • two independent reviews confirmed exact moved-body equivalence, compile inclusion, path/lifecycle semantics, and encoding/line limits

Closes #10642

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings August 18, 2026 22:14
@Evangelink Amaury Levé (Evangelink) added the state/needs-review Awaiting review from the team. label Aug 18, 2026

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

Splits AzureDevOpsSummaryReporter into focused partial-class modules without changing behavior.

Changes:

  • Retains identity, dependencies, and metadata in the core file.
  • Moves lifecycle/path handling and Markdown rendering into dedicated files.
  • Keeps all partial files below 300 lines with UTF-8 BOM.
Show a summary per file
File Description
AzureDevOpsSummaryReporter.cs Defines the core partial class and metadata.
AzureDevOpsSummaryReporter.Lifecycle.cs Contains session lifecycle and path-resolution logic.
AzureDevOpsSummaryReporter.Markdown.cs Contains Markdown generation and formatting helpers.

Review details

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state/needs-review Awaiting review from the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[file-diet] Refactor AzureDevOpsSummaryReporter.cs (607 lines) into focused modules

3 participants