Skip to content

Adding default provider and options provider pattern for platform users - #110

Merged
djpugh merged 1 commit into
mainfrom
feature/options-provider
Aug 9, 2026
Merged

Adding default provider and options provider pattern for platform users#110
djpugh merged 1 commit into
mainfrom
feature/options-provider

Conversation

@djpugh

@djpugh djpugh commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • [*] Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Have you updated the appropriate files

Please check that you have (if appropriate):

  • [*] Updated the documentation
  • Updated README.md
  • [*] Executed pre-commit run --all-files with no errors
  • [*] The change is fully covered by automated unit tests

What is the new behavior?

Optional default_provider and options_provider pattern (for platform users) to fetch runtime variables/settings

@djpugh
djpugh enabled auto-merge (squash) August 9, 2026 08:20
@sonarqubecloud

sonarqubecloud Bot commented Aug 9, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

☂️ Code Coverage

current status: ✅

Overall Coverage

Statements Covered Coverage Threshold Status
4174 3692 88% 80% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/nskit/client/field_models.py 100% 🟢
src/nskit/client/field_parser.py 96% 🟢
src/nskit/client/interactive.py 89% 🟢
src/nskit/mixer/components/recipe.py 92% 🟢
TOTAL 94% 🟢

updated for commit: 99e4f86 by action🐍

@djpugh
djpugh merged commit 5a25d4f into main Aug 9, 2026
63 checks passed
@djpugh
djpugh deleted the feature/options-provider branch August 9, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant