-
Notifications
You must be signed in to change notification settings - Fork 1
docs: add guide for optional output-format dependencies and strict JSON consumer validation #350
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
1b85015
89e9908
214a9d2
a02a295
445c02c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -37,3 +37,12 @@ For an optional polished human table, pass `rich=True` to `render_records()`. | |
| Rich is consulted only for interactive `text`; all redirected and structured | ||
| formats retain the rules above and fall back to the built-in renderer if Rich | ||
| is unavailable or fails. | ||
|
|
||
| ## Optional output-format dependencies | ||
|
|
||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Doc accuracy: this table says a missing PyYAML surfaces a raw
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed — the table now shows the actual wrapped message ( |
||
| Some output formats require additional dependencies. Install the corresponding extra to use them. | ||
|
|
||
| | Format | Extra | Install command | If missing | | ||
| |--------|-------|-----------------|------------| | ||
| | YAML | yaml | `pip install base-cli[yaml]` | `OutputFormatError: PyYAML is required for YAML output. Install the optional dependency with python -m pip install 'base-cli[yaml]'.` | | ||
| | Rich text (when `rich=True` is used in `render_records()`) | rich | `pip install base-cli[rich]` | Falls back to the built-in text renderer (no error, but output lacks Rich styling) | | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cleanup: the new "Contract fixtures and validator" section is a near-verbatim duplicate of the paragraph already present ~25 lines above it in the same file (both describe CI validating fixtures against packaged schemas via a Python validator and a Node.js reader). Consider merging to avoid having to update the same claim in two places.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed — the duplicate paragraph is gone in the latest commit (
docs: remove duplicate contract guidance). Thanks!