-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
docs: config overview #28035
docs: config overview #28035
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.
LGTM, waiting for @HonkingGoose suggestions.
You need to add this page here:
Line 6 in 4c8d3d4
- Configuration: |
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.
I'll let @rarkins make the changes proposed by the other reviewers first. 😉 That way I can start work from a better base for my PR that will target this PR.
Let me know when you're done making the changes, and this is ready for my in-depth review.
this new page still needs to be added to the |
@HonkingGoose this has now been stabilized enough that I think you can safely start a PR |
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
🎉 This PR is included in version 37.267.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Changes
Adds a new "Configuration Overview" page for user docs
Context
I don't think we've had a cohesive guide to config before, and some of the info in here perhaps is not mentioned anywhere else either.
To be merged after #27864 because I included info on Inherited config here
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: