-
Notifications
You must be signed in to change notification settings - Fork 2
Open
Labels
documentationImprovements or additions to documentationImprovements or additions to documentationenhancementNew feature or requestNew feature or requesttesting
Description
Overview
Track all work related to CLI documentation, test coverage, and help text organization.
Key Findings
- 6 undocumented flags in
docs/usage.md - 2 undocumented validation constraints
- 4 flags with incomplete test coverage
- Help text organization issues
- Limited short flag availability
Child Issues
P0 - Critical (Documentation)
- Document missing CLI flags in usage.md #495 - Document missing CLI flags in usage.md
- Document flag validation constraints #496 - Document flag validation constraints
P1 - High (Testing)
- Add integration test for --skip-pull flag #497 - Add integration test for --skip-pull flag
- Add tests for --allow-host-ports validation #498 - Add tests for --allow-host-ports validation
- Add tests for --proxy-logs-dir flag #499 - Add tests for --proxy-logs-dir flag
P2 - Medium (Enhancement)
- Improve help text organization and consistency #500 - Improve help text organization and consistency
P3 - Low
- Add short flags for frequently used options #501 - Add short flags for frequently used options
- Clarify --image-tag behavior with presets #502 - Clarify --image-tag behavior with presets
Implementation Order
Week 1 (Documentation):
Issue #495 (Doc flags) ────────┐
├──► Week 3 (Enhancement)
Issue #496 (Doc constraints) ──┘ │
├──► Issue #500 (Help text)
Week 2 (Testing): │
Issue #497 (skip-pull tests) ───────────►┤
Issue #498 (host-ports tests) ──────────►┤
Issue #499 (proxy-logs tests) ──────────►┘
Week 4 (Low Priority):
Issue #501 (Short flags) - Requires RFC
Issue #502 (Image-tag clarity)
Recommended PR Sequence
- PR 1: Issues Document missing CLI flags in usage.md #495 + Document flag validation constraints #496 + Clarify --image-tag behavior with presets #502 (all documentation, one PR)
- PR 2: Issue Add integration test for --skip-pull flag #497 (skip-pull integration tests)
- PR 3: Issues Add tests for --allow-host-ports validation #498 + Add tests for --proxy-logs-dir flag #499 (validation tests)
- PR 4: Issue Improve help text organization and consistency #500 (help text reorganization)
- PR 5: Issue Add short flags for frequently used options #501 (short flags - after RFC)
Backward Compatibility
All changes are backward compatible:
- Documentation updates only affect developer experience
- Test additions do not change behavior
- Help text changes do not affect CLI parsing
- Short flag additions are additive, not breaking
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
documentationImprovements or additions to documentationImprovements or additions to documentationenhancementNew feature or requestNew feature or requesttesting