Skip to content

docs: require README updates alongside CLI changes - #63

Merged
alexeyzimarev merged 1 commit into
mainfrom
docs/claude-md-readme-sync
May 13, 2026
Merged

alexeyzimarev merged 1 commit into
mainfrom
docs/claude-md-readme-sync

Conversation

@alexeyzimarev

Copy link
Copy Markdown
Member

Summary

Adds a CLAUDE.md note (under Common mistakes to avoid) instructing future contributors — human and AI — to update README.md in the same PR as any user-facing CLI change.

The trigger: AI-613 (PR #58) and AI-70 (PR #55) both shipped substantial CLI surface changes without touching README.md, requiring follow-up doc-only PRs (#60 and #61). The new bullet:

  • names the kinds of changes that require a README update (new command, new/renamed/removed flag, default-behavior shift, new prerequisite)
  • points at the two README sections that typically need touching (## Getting started quick-start and ## CLI commands per-command sections)
  • calls out that updating only src/Kapacitor.Core/Resources/help-*.txt isn't enough — --help output is internal, the README is the public docs.

Test plan

  • Diff reviewed — single bullet appended to existing list, no other changes.
  • Validate next CLI-touching PR includes README updates (the test of whether this note actually helps).

🤖 Generated with Claude Code

User-facing CLI changes (new commands, flags, behavior shifts) have
repeatedly landed without touching README.md, requiring follow-up
doc-only PRs (#60 for AI-613 history scope flags, #61 for AI-70
Codex hook surface). Adds a "Common mistakes to avoid" bullet
calling this out, naming the two README sections that typically
need updating, and pointing out that help-*.txt resources aren't a
substitute for the public README.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@qodo-code-review

Copy link
Copy Markdown

Review Summary by Qodo

Document requirement to sync README with CLI changes

📝 Documentation

Grey Divider

Walkthroughs

Description
• Adds guidance for CLI changes requiring README updates
• Specifies sections to update in README.md
• References past follow-up PRs caused by missing docs
• Clarifies help-*.txt files insufficient for public docs
Diagram
flowchart LR
  CLI["CLI Surface Changes"]
  README["README.md Updates"]
  SECTIONS["Quick-start & Commands"]
  CLI -- "must update" --> README
  README -- "check sections" --> SECTIONS
Loading

Grey Divider

File Changes

1. CLAUDE.md 📝 Documentation +1/-0

Add CLI-to-README sync requirement guidance

• Adds new bullet point to "Common mistakes to avoid" section
• Documents requirement to update README.md alongside CLI changes
• Specifies which README sections need updates (Getting started, CLI commands)
• Clarifies that help-*.txt resources are insufficient for public documentation
• References previous follow-up PRs (#60, #61) caused by missing updates

CLAUDE.md


Grey Divider

Qodo Logo

@alexeyzimarev
alexeyzimarev merged commit 4233d33 into main May 13, 2026
3 checks passed
@alexeyzimarev
alexeyzimarev deleted the docs/claude-md-readme-sync branch May 13, 2026 14:11
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.

1 participant