engine: add bind-create-src mount option#24140
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Documentation Review Summary
Found 2 medium-severity issues in the new content documenting the bind-create-src option:
- Terminology inconsistency: Using "source directory" vs "source path" inconsistently
- Misleading example: Using
/dev/mydirwhich suggests creating directories in a system-reserved path
Both issues are in the newly added content and should be addressed for clarity and accuracy.
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
There was a problem hiding this comment.
Review Summary
Assessment: 🟡 NEEDS ATTENTION
Found 1 issue in the new bind-create-src option documentation.
Summary
The PR successfully documents the new bind-create-src mount option, but there's one terminology inconsistency that should be corrected for clarity.
Issue:
- MEDIUM severity: Inconsistent terminology in the option description (uses "daemon" instead of "host")
Details
The changes correctly add "By default," qualifiers to explain the new behavior option, and the command syntax examples are accurate. The table entry for the new option needs a terminology fix to maintain consistency with the rest of the document.
Description
Adds a new option for
--mount type=bindRelated issues or tickets