Extend and clean up the extension section - #1068
Conversation
✅ Deploy Preview for gardener-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
🚧 Files skipped from review as they are similar to previous changes (6)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe documentation configuration updates navigation titles, extension ordering, provider and extension names, numeric weights, and Gardener extension entries. ChangesDocumentation catalog
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This PR updates extension documentation and sidebar naming without changing product behavior or runtime systems; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (7 skipped: 7 unsupported.) Full details: Description checkExplanation The description includes the PR category, purpose, detailed changes, related issue, and reviewer context. The issue is marked as related rather than using the template's exact “Fixes #” format, but the required context is present. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
b0b0f66 to
5817737
Compare
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
FalcoExtension.md seems to be an enhancement proposal. Moreover, it contains a broken image path, which leads to an error when running Docforge Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
5817737 to
ffaaabe
Compare
Sitemap structure diff vs
|
| count | |
|---|---|
| ➕ added pages | 0 |
| ➖ removed pages | 0 |
First entries (full diff in the run summary & artifact):
ADDED (0):
(none)
REMOVED (0):
(none)
A rename appears as one removed (old path) + one added (new path).
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
ffaaabe to
9212f5f
Compare
|
The build is failing because there are unescaped angle brackets in one of the pulled documents for the vSphere extension: https://github.com/gardener-attic/gardener-extension-provider-vsphere/blob/642bbe1c04fb7e2f9f767cad5b87c42b16057af8/docs/usage/prepare-vsphere.md?plain=1#L237. I'll remove the extension from the current PR and add it as soon as the problem is fixed in the source repository. |
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
|
Blocked by #1071 |
@BoHristova, I'll convert the PR to a draft and put it on hold until the issue is resolved, since the changes aren't visible in the Netlify preview deployment. /hold |
How to categorize this PR?
/kind enhancement cleanup
What this PR does / why we need it:
This PR brings the List of Extensions section of the documentation up to date with the Known Extension Implementations list maintained in
gardener/gardener.Еxtensions added:
Infrastructure providers
KubeVirtthe extension isn't actively maintained anymorevSphere(had to be excluded from the current PR because unescaped angle brackets in the source docs were causing a build error; will be added in a different PR after the problem is fixed)the extension is not maintained anymoreOperating system
Others
Sidebar improvements
weightfrontmatter to all Others entries to enforce consistent alphabetical orderWhich issue(s) this PR fixes:
Related to #1023
Special notes for your reviewer:
Summary by CodeRabbit
Documentation