docs: mention Docker CLI for Colima installs#6078
Conversation
Signed-off-by: WilliamK112 <164879897+WilliamK112@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughDocumentation updates across CI platform matrix config and three MDX docs files add guidance that macOS Homebrew Colima users must also install the Docker CLI separately ( ChangesColima Docker CLI Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
✨ Thanks for the fix. Adding the Docker CLI clarification for Colima installs and updating the platform matrix together keeps docs and generated tables in sync. Related open issues: |
Summary
ci/platform-matrix.jsonand the generated platform-support tables togetherFixes #6028
Validation
git diff --checkPYTHONUTF8=1 python scripts/generate-platform-docs.py --checkrg -n "brew install colima docker|brew install docker|Homebrew Colima|Docker CLI separately|colima start --cpu 4 --memory 8" ci docsNotes
npx vitest run --project integration test/generate-platform-docs.test.tsruns with a temporarypython3.exeshim and reaches the test body; 15/18 tests pass, while 3 existing assertions assume Linux stdout newlines or POSIX temp paths. This PR only changes docs/matrix content.Summary by CodeRabbit
docker info.