Skip to content

Refactor WindowManager into partial classes#109

Merged
PrzemyslawKlys merged 1 commit intov2-speedygonzalesfrom
refactor-project-for-maintainability-and-structure
Jul 10, 2025
Merged

Refactor WindowManager into partial classes#109
PrzemyslawKlys merged 1 commit intov2-speedygonzalesfrom
refactor-project-for-maintainability-and-structure

Conversation

@PrzemyslawKlys
Copy link
Copy Markdown
Member

Summary

  • split monolithic WindowManager class into partial classes WindowManager.Actions and WindowManager.Layout
  • convert WindowManager to file-scoped namespace

Testing

  • dotnet build Sources/DesktopManager.sln -c Release
  • dotnet test Sources/DesktopManager.sln -c Release (fails: Could not find 'mono' host)
  • pwsh -NoLogo -NoProfile -Command ./DesktopManager.Tests.ps1 (fails: cmdlets not recognized)

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

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 10, 2025

Codecov Report

Attention: Patch coverage is 45.75163% with 83 lines in your changes missing coverage. Please review.

Project coverage is 53.23%. Comparing base (84fee87) to head (c6f9dc4).
Report is 8 commits behind head on v2-speedygonzales.

Files with missing lines Patch % Lines
Sources/DesktopManager/WindowManager.Actions.cs 40.00% 56 Missing and 7 partials ⚠️
Sources/DesktopManager/WindowManager.Layout.cs 58.33% 14 Missing and 6 partials ⚠️
Additional details and impacted files
@@                  Coverage Diff                  @@
##           v2-speedygonzales     #109      +/-   ##
=====================================================
+ Coverage              51.64%   53.23%   +1.59%     
=====================================================
  Files                     28       31       +3     
  Lines                   1340     1422      +82     
  Branches                 206      222      +16     
=====================================================
+ Hits                     692      757      +65     
- Misses                   573      583      +10     
- Partials                  75       82       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@PrzemyslawKlys PrzemyslawKlys merged commit 0494288 into v2-speedygonzales Jul 10, 2025
8 of 9 checks passed
@PrzemyslawKlys PrzemyslawKlys deleted the refactor-project-for-maintainability-and-structure branch July 10, 2025 20:20
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