Skip to content

Move @SafeDIConfiguration docs to its own section under Macros#225

Merged
dfed merged 1 commit intomainfrom
dfed/safediconfiguration-docs
Apr 7, 2026
Merged

Move @SafeDIConfiguration docs to its own section under Macros#225
dfed merged 1 commit intomainfrom
dfed/safediconfiguration-docs

Conversation

@dfed
Copy link
Copy Markdown
Owner

@dfed dfed commented Apr 7, 2026

Summary

  • @SafeDIConfiguration was only documented inside the "Mock generation" section, making it appear mock-specific
  • It configures three distinct concerns (additionalImportedModules, additionalDirectoriesToInclude, mockConditionalCompilation), so it belongs alongside the other macro docs

What changed

  • Added ### @SafeDIConfiguration section under ## Macros (after @Received, before ## Delayed instantiation)
  • Each property gets its own #### subsection with type, default, and description matching the /// doc comments in source
  • Updated the macro table link from #enabling-mock-generation to #safediconfiguration
  • Slimmed the mock generation section to reference the new section for mockConditionalCompilation instead of duplicating the full configuration example

Test plan

  • Documentation-only change
  • Build succeeds

🤖 Generated with Claude Code

@dfed dfed force-pushed the dfed/safediconfiguration-docs branch from 01acb1d to 1fc54de Compare April 7, 2026 05:03
@dfed dfed marked this pull request as ready for review April 7, 2026 05:07
The @SafeDIConfiguration macro was only documented inside the mock
generation section, making it appear mock-specific. It configures
three distinct concerns (imports, directories, mock compilation), so
it belongs alongside the other macro docs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dfed dfed force-pushed the dfed/safediconfiguration-docs branch from 1fc54de to fb1e218 Compare April 7, 2026 05:09
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d8f981c) to head (fb1e218).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #225   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           40        40           
  Lines         6128      6128           
=========================================
  Hits          6128      6128           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dfed dfed merged commit 28d2cd0 into main Apr 7, 2026
19 checks passed
@dfed dfed deleted the dfed/safediconfiguration-docs branch April 7, 2026 05: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.

1 participant