docs: fix duplicate headings in overlayfs driver page#24428
docs: fix duplicate headings in overlayfs driver page#24428dvdksn wants to merge 1 commit intodocker:mainfrom
Conversation
Rename the second "Image and container layers on-disk" section heading to "Image and container layers on-disk (legacy overlay driver)" to distinguish it from the overlay2 section above it. Fixes docker#18191. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
bot
added
area/engine
![docker-agent[bot]](https://avatars.githubusercontent.com/in/2787757?s=60&v=4){kind=small}
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This PR successfully fixes the duplicate heading issue reported in #18191. The change is clean and appropriate:
✅ Heading disambiguation: The second "Image and container layers on-disk" heading (line 227) is now clearly labeled as "(legacy overlay driver)" to distinguish it from the first section about overlay2
✅ Minimal scope: The change only affects what's necessary to fix the duplicate heading issue
✅ Formatting cleanup: The indentation changes to the definition list are valid Markdown formatting normalization
The renamed heading makes it clear to readers that the second section describes the legacy overlay driver's on-disk structure, improving navigation and reducing confusion.
No issues found. Ready to merge.
1 participant
Summary
overlayfs-driver.mdpage had two sections with identical headings:### Image and container layers on-diskoverlay2driver's layer layout on diskoverlaydriver's layer layout on disk### Image and container layers on-disk (legacy overlay driver)to make the distinction clearTest plan
/engine/storage/drivers/overlayfs-driver/🤖 Generated with Claude Code