Skip to content

Conversation

@jreidinger
Copy link
Contributor

A bit of details. If allOf is used, then it is correct also if just single one is used, but intention is to express that one of given parameters is specified or none.

A bit of details. If allOf is used, then it is correct also if just
single one is used, but intention is to express that one of given
parameters is specified or none.
Copy link
Contributor

@dgdavid dgdavid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving it assuming manual testing, although I still thinking would be better to have unit testing to ensure the expected behavior.

@jreidinger jreidinger merged commit 3024cfc into master Apr 4, 2025
7 checks passed
@jreidinger jreidinger deleted the fix_schema branch April 4, 2025 14:12
@imobachgs imobachgs mentioned this pull request Apr 22, 2025
imobachgs added a commit that referenced this pull request Apr 22, 2025
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.

3 participants