Skip to content

fix: replace stale --json refs in markdown footer#24

Merged
DTTerastar merged 1 commit into
mainfrom
fix/stale-json-doc-strings
Apr 25, 2026
Merged

fix: replace stale --json refs in markdown footer#24
DTTerastar merged 1 commit into
mainfrom
fix/stale-json-doc-strings

Conversation

@DTTerastar

Copy link
Copy Markdown
Collaborator

Summary

The --json flag was removed in 054064e in favor of --format json, but two markdown footer strings (renderServings, renderNutrition) still pointed users at the dead flag — an LLM or human following the suggestion verbatim hits error: unknown flag: --json.

Closes #20

Test plan

  • go build ./... clean
  • go vet ./... clean
  • grep -rn "\-\-json" cmd/ returns nothing

🤖 Generated with Claude Code

The --json flag was removed in 054064e in favor of --format json,
but the renderServings and renderNutrition footer strings still
told users to "use --json for the full row".

Closes #20

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@DTTerastar
DTTerastar merged commit 3792cdc into main Apr 25, 2026
1 check passed
@DTTerastar
DTTerastar deleted the fix/stale-json-doc-strings branch April 26, 2026 00:24
Terastar-Paperclip added a commit that referenced this pull request May 27, 2026
Moves the crono-export worked example out of quantcli/common (where it
was misfiled in PR #24) and into this repo where it belongs — next to
the binary it documents. Co-references in GETTING_STARTED.md will be
updated in a follow-up PR against quantcli/common.

Co-authored-by: LeadGoEngineer <noreply@paperclip.ing>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Markdown renderers emit 'use --json for the full row' but --json was removed in favor of --format json

1 participant