Fix ConceptLink warnings in virt/about_virt documentation#111741
Fix ConceptLink warnings in virt/about_virt documentation#111741abrennan89 wants to merge 1 commit into
Conversation
- Restore [role="_abstract"] attributes that were incorrectly removed - Remove blank line between Additional resources heading and list - Update link text to match actual document titles - Remove duplicate xref entry Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
All review comments addressed:
Ready for another review! |
- Add quotation marks around 'Virtualization limits for Red Hat Enterprise Linux with KVM' in module - Update link text to match actual document titles: - 'OVN-Kubernetes' → 'About the OVN-Kubernetes network plugin' - 'Compliance Operator' → 'Understanding the Compliance Operator' - Move KVM limits link to Additional resources in assembly Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
Final review suggestions applied: ✅ modules/virt-tested-maximums.adoc
✅ virt/about_virt/about-virt.adoc
✅ virt/about_virt/virt-supported-limits.adoc
All link text now matches actual document titles. Ready for final review! |
Move links and cross-references to Additional resources sections in assembly files. Remove links from module and snippet files to comply with documentation standards. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Address PR review comments for openshift#111741 - Restore [role="_abstract"] attributes that were incorrectly removed - Remove blank line between Additional resources heading and list - Update link text to match actual document titles - Remove duplicate xref entry Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Apply final review suggestions for openshift#111741 - Add quotation marks around 'Virtualization limits for Red Hat Enterprise Linux with KVM' in module - Update link text to match actual document titles: - 'OVN-Kubernetes' → 'About the OVN-Kubernetes network plugin' - 'Compliance Operator' → 'Understanding the Compliance Operator' - Move KVM limits link to Additional resources in assembly Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> fix conditional links
f1a7ddc to
c758d6d
Compare
|
Fixed AsciiDocDITA.AssemblyContents error and remaining issues: Issues addressed:
Result:
All content with removed links has been moved to the appropriate module where it belongs. Ready for review! |
Move links and cross-references to Additional resources sections in assembly files. Remove links from module and snippet files to comply with documentation standards. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Address PR review comments for openshift#111741 - Restore [role="_abstract"] attributes that were incorrectly removed - Remove blank line between Additional resources heading and list - Update link text to match actual document titles - Remove duplicate xref entry Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Apply final review suggestions for openshift#111741 - Add quotation marks around 'Virtualization limits for Red Hat Enterprise Linux with KVM' in module - Update link text to match actual document titles: - 'OVN-Kubernetes' → 'About the OVN-Kubernetes network plugin' - 'Compliance Operator' → 'Understanding the Compliance Operator' - Move KVM limits link to Additional resources in assembly Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> fix conditional links Fix AsciiDocDITA.AssemblyContents error by moving content to module - Move OVN-Kubernetes, Compliance Operator, and Red Hat Ecosystem Catalog content from assembly to module - Assembly should only contain abstract, includes, and Additional resources - Fix ID to use {context} variable instead of hardcoded value - Resolves AsciiDocDITA.AssemblyContents: 'Content other than additional resources cannot follow include directives' Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
c83ece1 to
eb96694
Compare
|
@abrennan89: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Summary
This PR fixes AsciiDocDITA.ConceptLink warnings in the virt/about_virt documentation by:
Preview links:
Version:
4.20+
Jira:
TBA
Related
Part of splitting PR #111736 into smaller, subdirectory-focused PRs for easier review.
🤖 Generated with Claude Code