Skip to content

feat: add scoped settings context manager#3440

Open
PragnyaKhandelwal wants to merge 1 commit into
docling-project:mainfrom
PragnyaKhandelwal:feature/scoped-settings-context-manager
Open

feat: add scoped settings context manager#3440
PragnyaKhandelwal wants to merge 1 commit into
docling-project:mainfrom
PragnyaKhandelwal:feature/scoped-settings-context-manager

Conversation

@PragnyaKhandelwal
Copy link
Copy Markdown

Summary

  • Add a scoped settings context manager for temporary perf/debug/inference overrides
  • Add a defaults accessor to create a fresh AppSettings
  • Add a regression test to verify settings are restored after the scoped block exits

Issue resolved by this Pull Request:
Resolves #3417

Checklist:

  • Documentation has been updated, if necessary.
  • Examples have been added, if necessary.
  • Tests have been added, if necessary.

Signed-off-by: Pragnya <prag1704@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

DCO Check Passed

Thanks @PragnyaKhandelwal, all your commits are properly signed off. 🎉

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented May 13, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

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.

Feature: scoped context manager for AppSettings (perf/debug/inference) — replace process-wide singleton mutation

1 participant