Skip to content

Restore dropped XML docs in AppSettingsService#352

Merged
erikdarlingdata merged 2 commits into
devfrom
chore/restore-appsettings-docs
Jun 2, 2026
Merged

Restore dropped XML docs in AppSettingsService#352
erikdarlingdata merged 2 commits into
devfrom
chore/restore-appsettings-docs

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • Re-add the XML summary for AccuracyRatioDivergenceLimit that was lost in Feature/settings #351 when the property was moved into the App State block.
  • Document that Load() returns the shared cached instance so callers don't accidentally mutate it and corrupt the cache for everyone else.

Test plan

  • Docs-only change, no behavior impact.
  • CI green.

🤖 Generated with Claude Code

erikdarlingdata and others added 2 commits May 27, 2026 19:48
- Re-add summary for AccuracyRatioDivergenceLimit that was lost when the
  property moved into the App State block.
- Document that Load() returns the shared cached instance so callers know
  not to mutate it and corrupt the cache for everyone else.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Pre-existing dead using surfaced after the doc edits — no LINQ methods
are used in this file.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit b183082 into dev Jun 2, 2026
2 checks passed
@erikdarlingdata erikdarlingdata deleted the chore/restore-appsettings-docs branch June 2, 2026 23:39
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