Skip to content

refactor: cycle 18 Infrastructure - remove dead dependency and constructor#638

Merged
rygel merged 2 commits into
developfrom
feat/cycle-18-infrastructure
Jun 12, 2026
Merged

refactor: cycle 18 Infrastructure - remove dead dependency and constructor#638
rygel merged 2 commits into
developfrom
feat/cycle-18-infrastructure

Conversation

@rygel

@rygel rygel commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Cycle 18 Lean Code Sweep (Infrastructure tasks).

Changes:

  • Remove dead System.Drawing.Common NuGet reference from Infrastructure.csproj (zero usage)
  • Remove dead default constructor in MonitorProcessService that bypassed DI

Verified with dotnet build + dotnet test.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Large PR Detected

This PR contains 2483 lines changed across 87 files.

Details:

  • 📁 Files changed: 87
  • ➕ Insertions: 691
  • ➖ Deletions: 1792
  • 📊 Total: 2483 lines

Recommendation:
Consider splitting this PR into smaller, focused changes to make review easier and faster. PRs under 500 lines are ideal for thorough code review.


This is an automated message. Feel free to ignore if this PR is intentionally large.

@rygel
rygel force-pushed the feat/cycle-18-infrastructure branch from d928304 to 3aa31df Compare June 12, 2026 10:39
@rygel
rygel force-pushed the feat/cycle-18-infrastructure branch from 3aa31df to 4deb894 Compare June 12, 2026 12:03
@rygel
rygel merged commit 1724f3a into develop Jun 12, 2026
9 of 10 checks passed
@rygel
rygel deleted the feat/cycle-18-infrastructure branch June 12, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants