Skip to content

Show SolutionConfiguration property when multiple distinct values exist #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Copilot
Copy link

@Copilot Copilot AI commented Jun 9, 2025

Previously, projects that differed only by the SolutionConfiguration property were hard to diagnose because this property was treated as common and elided entirely from the output, even when different values existed across builds.

This change enhances the logger to:

  1. Track distinct SolutionConfiguration values across all project builds
  2. Display SolutionConfiguration property values when multiple distinct values are detected (instead of eliding them with {elided})
  3. Preserve existing behavior when only one SolutionConfiguration value exists

Example

Before: Projects with different SolutionConfiguration values would show no distinguishing properties

Project 3: test.csproj + <SolutionConfiguration = {elided}>
Project 4: test.csproj + <SolutionConfiguration = {elided}>

After: Projects with different SolutionConfiguration values now show the actual values

Project 3: test.csproj + <SolutionConfiguration = Debug>
Project 4: test.csproj + <SolutionConfiguration = Release>

Additional Fix

Fixed a bug in GlobalPropertyValue.CompareTo() method that was comparing Value.CompareTo(Value) instead of Value.CompareTo(other.Value), which could affect property sorting.

Testing

  • ✅ Multiple SolutionConfiguration values are displayed with their actual values
  • ✅ Single SolutionConfiguration value continues to be elided (preserving existing behavior)
  • ✅ Other properties (CurrentSolutionConfigurationContents, etc.) remain unchanged
  • ✅ Original example builds continue to work as expected

Fixes #13.


💡 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.

Co-authored-by: rainersigwald <3347530+rainersigwald@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Projects that differ only by the SolutionConfiguration property are hard to diagnose Show SolutionConfiguration property when multiple distinct values exist Jun 9, 2025
@Copilot Copilot AI requested a review from rainersigwald June 9, 2025 22:14
Copilot finished work on behalf of rainersigwald June 9, 2025 22: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.

Projects that differ only by the SolutionConfiguration property are hard to diagnose
2 participants