Add variable registration to ConfigVariableReader#9
Merged
prachigauriar merged 1 commit intomainfrom Mar 6, 2026
Merged
Conversation
- Add RegisteredConfigVariable, a struct that captures a variable's
key, default content, secrecy, and metadata for storage in
homogeneous collections
- Add encode closure to ConfigVariableContent for converting values
back to ConfigContent, enabling default value encoding at
registration time
- Add encodeToContent on CodableValueRepresentation for converting
encoded Data into the appropriate ConfigContent
- Add ConfigVariableReader.register, which encodes and stores a
variable's registration with duplicate-key detection and logging
- Split ConfigVariableReaderTests into per-concern test files:
scalar, array, raw representable, config expression, data
representation, codable, and registration
- Extract shared test mocks and metadata keys into Testing Support
Code Coverage ReportOverall Coverage: 99.57% (1,867 of 1,875) DevConfiguration: 99.57% (1,867 of 1,875)
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
key, default content, secrecy, and metadata for storage in
homogeneous collections
back to ConfigContent, enabling default value encoding at
registration time
encoded Data into the appropriate ConfigContent
variable's registration with duplicate-key detection and logging
scalar, array, raw representable, config expression, data
representation, codable, and registration