Conversation
Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
Performance Regression Reportscommit: c75e98d There are no regressions. |
Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
… refactor test Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
…tertools join, new detection spec Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
… detect boolean flags Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the project’s CLI documentation pipeline by generating USAGE.md and help snapshots directly from clap’s introspection APIs, and adds tests to keep those generated artifacts in sync with the actual CLI.
Changes:
- Add
usage_mdrenderer (src/usage_md.rs) and a dedicatedpdu-usage-mdbinary to generateUSAGE.md. - Gate/export the new module behind
cfg(feature = "cli")to fix doc builds without CLI features. - Add snapshot-style tests and generation script updates for
exports/*.helpandUSAGE.md, and linkUSAGE.mdfrom the README.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/sync_help.rs | Adds snapshot tests to detect stale exports/*.help and USAGE.md. |
| src/usage_md.rs | Implements Markdown rendering for CLI usage/options/examples via clap introspection. |
| src/lib.rs | Gates and exports usage_md behind the cli feature. |
| generate-completions.sh | Extends generator script to also emit long/short help and USAGE.md. |
| exports/short.help | Adds generated short help snapshot. |
| exports/long.help | Adds generated long help snapshot. |
| cli/usage_md.rs | Adds pdu-usage-md binary entrypoint to print rendered Markdown. |
| USAGE.md | Adds regenerated CLI usage reference page. |
| README.md | Links to USAGE.md from the README. |
| Cargo.toml | Registers the new pdu-usage-md binary (feature-gated). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| //! | ||
| //! If the tests fail, run `./generate-completions.sh` on the root of the repo to update the help files. | ||
|
|
||
| // Since the CLI in Windows look a little different, and I am way too lazy to make two versions |
There was a problem hiding this comment.
Grammar: "the CLI in Windows look" should be "the CLI in Windows looks".
| // Since the CLI in Windows look a little different, and I am way too lazy to make two versions | |
| // Since the CLI in Windows looks a little different, and I am way too lazy to make two versions |
Fixes #323
Fixes #54