New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: resolve mismatch with types declaration and docs for resourceQuery #13296
Conversation
For maintainers only:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a test case -> test\configCases\rule-set
schemas/WebpackOptions.json
Outdated
"description": "Logical AND.", | ||
"oneOf": [ | ||
{ | ||
"$ref": "#/definitions/RuleSetConditions" | ||
"$ref": "#/definitions/RuleSetCondition" | ||
} | ||
] | ||
}, | ||
"not": { | ||
"description": "Logical NOT.", | ||
"oneOf": [ | ||
{ | ||
"$ref": "#/definitions/RuleSetConditions" | ||
"$ref": "#/definitions/RuleSetCondition" | ||
} | ||
] | ||
}, | ||
"or": { | ||
"description": "Logical OR.", | ||
"oneOf": [ | ||
{ | ||
"$ref": "#/definitions/RuleSetConditions" | ||
"$ref": "#/definitions/RuleSetCondition" | ||
} | ||
] | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you put the whole object with and, or, not into a separate definition RuleSetLogicalConditions
.
This helps in generating more readable typings.
Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon. |
Thanks |
Resolves #13284
I followed the suggestion provided and made changes accordingly
What kind of change does this PR introduce?
This is a bugfix
Did you add tests for your changes?
I updated the snapshot tests
Does this PR introduce a breaking change?
No, it does not
What needs to be documented once your changes are merged?
I don't think any changes need to be made to the documentation