Conversation
dvdksn
commented
Feb 19, 2026
dvdksn
commented
- sandboxes: note v0.12 prereq for opencode, shell, multi-workspace
- sandboxes: fix incorrect descriptions for --pull-template
- sandboxes: clarify security implications for auth methods
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
![docker-agent[bot]](https://avatars.githubusercontent.com/in/2787757?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Summary
Nice cleanup! The PR successfully:
- Adds v0.12 version requirements for new features (opencode, shell, multi-workspace)
- Fixes
--pull-templateflag descriptions to be more accurate - Clarifies security implications for authentication methods
Found one minor style improvement opportunity regarding redundant phrasing. Otherwise the changes look good!
| > version 0.12). For Docker Desktop 4.58–4.60, substitute | ||
| > `--pull-template=never` with `--load-local-template`. | ||
| > version 0.12). On Docker Desktop 4.58–4.60, use `--load-local-template` | ||
| > to use a locally-built template image. |
There was a problem hiding this comment.
Minor style improvement
The phrase "use --load-local-template to use a locally-built template image" has redundant usage of "use...to use". Consider simplifying:
Option 1 (recommended):
On Docker Desktop 4.58–4.60, use
--load-local-templatefor locally-built template images.
Option 2:
On Docker Desktop 4.58–4.60,
--load-local-templateworks with locally-built template images.
This aligns with the Docker style guide's emphasis on conciseness.
5 participants
