Skip to content

Improvements to the DI overview remarks - #37663

Merged
guardrex merged 3 commits into
mainfrom
guardrex-patch-4
Sep 21, 2026
Merged

guardrex merged 3 commits into
mainfrom
guardrex-patch-4

Conversation

@guardrex

@guardrex guardrex commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

Fixes #28220

Thanks @m-brooks! 🚀

... and sorry for the delay in us reaching your issue. We've been very busy around here ..... for years! ⛏️😅😆

Tom ... Found this DI issue while surveying issues for the Blazor work. See if fleshing out these remarks looks good.


Internal previews

File Preview link
aspnetcore/fundamentals/dependency-injection.md Learn preview

Build report

This comment was marked as resolved.

Clarified the unit testing challenges related to direct dependencies in dependency injection.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@guardrex
guardrex requested a review from tdykstra September 16, 2026 15:34

This comment was marked as resolved.

@guardrex

Copy link
Copy Markdown
Collaborator Author

Re-ping @tdykstra for review.

@tdykstra tdykstra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@guardrex
guardrex merged commit a3e2ecd into main Sep 21, 2026
6 checks passed
@guardrex
guardrex deleted the guardrex-patch-4 branch September 21, 2026 21:43
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.

Rationale needed for claims made by this page

3 participants