Skip to content

Fix Dependency injection analyzer that is supposed to come with the SDK - #320

Draft
justindbaur wants to merge 1 commit into
mainfrom
fix-dependency-injection-analyzer
Draft

Fix Dependency injection analyzer that is supposed to come with the SDK#320
justindbaur wants to merge 1 commit into
mainfrom
fix-dependency-injection-analyzer

Conversation

@justindbaur

Copy link
Copy Markdown
Member

🎟️ Tracking

📔 Objective

The DI analyzer is not working once I have bumped the server sdk to 1.6.0 in server. This registers them manually since msbuild sdks don't automatically add analyzers that are in their analyzers folder like normal nuget packages do.

🚨 Breaking Changes

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.73%. Comparing base (a22c13b) to head (aa660ae).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #320      +/-   ##
==========================================
+ Coverage   76.57%   76.73%   +0.15%     
==========================================
  Files          61       61              
  Lines        1887     1887              
  Branches      229      229              
==========================================
+ Hits         1445     1448       +3     
+ Misses        360      358       -2     
+ Partials       82       81       -1     

☔ View full report in Codecov by Harness.
📢 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.

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.

1 participant