Skip to content

fix optional configs handling#716

Draft
lublak wants to merge 2 commits intowarpstreamlabs:mainfrom
lublak:fix_optional_handling
Draft

fix optional configs handling#716
lublak wants to merge 2 commits intowarpstreamlabs:mainfrom
lublak:fix_optional_handling

Conversation

@lublak
Copy link
Copy Markdown
Contributor

@lublak lublak commented Feb 23, 2026

first step to fix #654

@lublak lublak changed the title add optional tests fix optional configs handling Feb 23, 2026
@lublak
Copy link
Copy Markdown
Contributor Author

lublak commented Feb 24, 2026

@jem-davies @gregfurman
Please take a look at the tests to see if you agree.
I think a fixed test structure and fixed specifications for how optional elements are handled would be helpful.

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.

service.ParsedConfig Contains returns true for optional NewObjectField

1 participant