Skip to content

Ensure monitor lookup ignores case#97

Merged
PrzemyslawKlys merged 1 commit intov2-speedygonzalesfrom
update-string-comparison-to-ordinal-ignore-case
Jul 8, 2025
Merged

Ensure monitor lookup ignores case#97
PrzemyslawKlys merged 1 commit intov2-speedygonzalesfrom
update-string-comparison-to-ordinal-ignore-case

Conversation

@PrzemyslawKlys
Copy link
Copy Markdown
Member

Summary

  • fix monitor device matching to use StringComparison.OrdinalIgnoreCase
  • cover mixed case device IDs in monitor position tests

Testing

  • dotnet build Sources/DesktopManager.sln -c Release
  • dotnet test Sources/DesktopManager.Tests/DesktopManager.Tests.csproj -c Release --no-build --framework net8.0 --logger "console;verbosity=detailed"
  • pwsh -Command ./DesktopManager.Tests.ps1 (fails: The term 'Save-DesktopWindowLayout' is not recognized)

https://chatgpt.com/codex/tasks/task_e_686d32241078832e920a962fbbe0eeb1

@PrzemyslawKlys PrzemyslawKlys merged commit a896e52 into v2-speedygonzales Jul 8, 2025
7 checks passed
@PrzemyslawKlys PrzemyslawKlys deleted the update-string-comparison-to-ordinal-ignore-case branch July 8, 2025 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant