Consolidate command-line argument tables into single unified reference and add --result alias#72
Conversation
… and guide - Replace fragmented Global Options/Lint Mode/Capture Mode/Publish Mode sub-sections (each with their own tables) with a single unified options table per document - Group header rows use bold labels: General, Lint Mode, Capture Mode, Publish Mode, Self-Validation - Include usage synopsis block in README.md Command-Line Options section - Remove per-command options tables from guide.md command sub-sections - Add --validate and --results to Self-Validation group in both documents - Ensure --report-depth description is consistent (includes min/max range) in both files Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
|
@copilot Excellent, now while we're at it, can you add "--result" as an alias for "--results". Don't bother adding "--result" to the README.md or user guide, as it's just legacy argument support. We want new users to see the correct "--results", but silently handle "--result" and possibly add a test to make sure it's supported. |
Agent-Logs-Url: https://github.com/demaconsulting/VersionMark/sessions/2f3ce4c7-5d91-4c45-baaf-e5e1a71678eb Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Done in b5d6f4f. Added |
There was a problem hiding this comment.
Pull request overview
This PR consolidates command-line option documentation into unified tables in the README and User Guide, and adds a legacy --result alias for --results in the CLI argument parser with a corresponding unit test.
Changes:
- Consolidate command-line options into a single grouped table in
README.mdanddocs/user_guide/guide.md. - Add
--resultas an alias for--resultsinContextargument parsing. - Add a unit test verifying the
--resultalias setsResultsFile.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| test/DemaConsulting.VersionMark.Tests/Cli/ContextTests.cs | Adds coverage for the legacy --result alias. |
| src/DemaConsulting.VersionMark/Cli/Context.cs | Extends argument parsing to accept --result as an alias for --results. |
| docs/user_guide/guide.md | Replaces per-command option tables with a single grouped options table. |
| README.md | Replaces multiple options sections with a unified grouped options table and usage synopsis. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
The README and User Guide split command-line arguments into separate markdown sections per mode (Global Options, Lint Mode, Capture Mode, Publish Mode), making it harder to see all options at a glance. Both docs now present all arguments in one unified table with inline group labels — matching how the CLI help message organises them.
README.md (
## Command-Line Options):### Global Options,### Lint Mode,### Capture Mode,### Publish Modesub-sections**General**,**Lint Mode**,**Capture Mode**,**Publish Mode**,**Self-Validation**)docs/user_guide/guide.md (
# Command-Line Reference):### Options/### Lint Options/### Publish Optionstables## Optionswith the same unified table structure## Lint Mode,## Capture Mode,## Publish Mode) retained for synopsis, behavior, and examples — option tables removed to avoid redundancyLegacy
--resultalias (src/DemaConsulting.VersionMark/Cli/Context.cs):--resultas a silent alias for--resultsin the argument parser for backward compatibility--resultsContext_Create_ResultFlag_SetsResultsFiletest to verify the alias is acceptedType of Change
Related Issues
Pre-Submission Checklist
Before submitting this pull request, ensure you have completed the following:
Build and Test
dotnet build --configuration Releasedotnet test --configuration Releasedotnet run --project src/DemaConsulting.VersionMark --configuration Release --framework net10.0--no-build -- --validateCode Quality
dotnet format --verify-no-changesQuality Checks
Please run the following checks before submitting:
cspell "**/*.{md,cs}"markdownlint "**/*.md"yamllint .Testing
Documentation
Additional Notes
The
--resultalias is intentionally undocumented — it exists solely for legacy compatibility. The README and User Guide continue to show only--results.