-
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
base: main
Are you sure you want to change the base?
Changes from all commits
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 |
||
| 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]` | `ImportError: No module named '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.