Skip to content

Implement Results Class to Replace Message-Based Result Storage#42

Draft
Copilot wants to merge 6 commits into
masterfrom
copilot/fix-31
Draft

Implement Results Class to Replace Message-Based Result Storage#42
Copilot wants to merge 6 commits into
masterfrom
copilot/fix-31

Conversation

Copy link
Copy Markdown

Copilot AI commented May 25, 2025

This PR enhances the application's result handling by implementing a robust Results class that replaces the previous message-based approach for storing comparison results.

Changes Made

Results Class Enhancements

  • Made SourceAlias and TargetAlias properties public for better accessibility
  • Added structured result parsing in the constructor
  • Added utility methods:
    • GetSummary() - Provides a concise overview of comparison results
    • GetUnmatchedFilesReport() - Generates a formatted report of unmatched files
    • GetResultsAsCSV() - Returns the results in CSV format
    • SaveToFile() and SaveToCSV() - Save results directly to files

New ResultItem Class

  • Created a class to provide structured access to individual results
  • Parses raw strings into strongly-typed properties
  • Provides access to DTA settings, comparison counts, and file information
  • Makes it easier to work with results programmatically

UI Improvements

  • Updated Form1 to use the enhanced Results class
  • Added detailed results display with DTA statistics
  • Added summary information dialog
  • Added unmatched files reporting
  • Uses the Results class directly for file saving

Benefits

  • More structured approach to handling results
  • Better user feedback with detailed statistics
  • Results are now stored in a format that's easier to work with programmatically
  • Makes the codebase more maintainable and flexible

Fixes #31.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 5 commits May 25, 2025 18:10
Co-authored-by: crcrewso <6089951+crcrewso@users.noreply.github.com>
Co-authored-by: crcrewso <6089951+crcrewso@users.noreply.github.com>
Co-authored-by: crcrewso <6089951+crcrewso@users.noreply.github.com>
Co-authored-by: crcrewso <6089951+crcrewso@users.noreply.github.com>
Co-authored-by: crcrewso <6089951+crcrewso@users.noreply.github.com>
Copilot AI changed the title [WIP] Results Class Implement Results Class to Replace Message-Based Result Storage May 25, 2025
Copilot AI requested a review from crcrewso May 25, 2025 18:14
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.

Results Class

2 participants