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
feat(config): validate top-level options #28383
base: main
Are you sure you want to change the base?
Conversation
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.
We need to check that each of these changes is correct and won't have unintended side effects.
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.
looks good so far, will run against some of my repos to validate different configs
Also need to think how to handle |
For repo config options, anything valid in the top level is valid within onboardingConfig |
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.
LGTM so far
# Conflicts: # test/documentation.spec.ts
Changes
Adds
.
to allowableparents
field and then adds this check to config validation.Context
Closes #28300
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: