Skip to content

docs: require explicit DI imports in snippets - #404

Merged
thomhurst merged 1 commit into
mainfrom
issue-398-readme-using
Aug 27, 2026
Merged

docs: require explicit DI imports in snippets#404
thomhurst merged 1 commit into
mainfrom
issue-398-readme-using

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Summary

  • fix README AddStandardShield namespace import
  • remove hidden DI import from snippet verifier
  • add explicit DI imports to affected documentation fences

Test plan

  • dotnet build Kevlar.slnx -c Release -p:Version=0.0.398
  • pwsh scripts/Verify-DocSnippets.ps1 -PackagesPath artifacts/package/release -Version 0.0.398 -NoImplicitUsings
  • pwsh scripts/Verify-Docs.ps1
  • npm run build --prefix docs

Closes #398

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 44435a67-a578-480c-84ad-c6824ae56c1f


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thomhurst
thomhurst merged commit a688ee1 into main Aug 27, 2026
7 checks passed
@thomhurst
thomhurst deleted the issue-398-readme-using branch August 27, 2026 19:59
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.

docs: fix README AddStandardShield using; stop Verify-DocSnippets injecting Microsoft.Extensions.DependencyInjection globally

1 participant