Skip to content

docs: keep the root README focused on CLI behavior - #975

Merged
sdairs merged 5 commits into
codex/focused-834from
codex/readme-cli-scope-cleanup
Sep 17, 2026
Merged

sdairs merged 5 commits into
codex/focused-834from
codex/readme-cli-scope-cleanup

Conversation

@sdairs

@sdairs sdairs commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

Keep the root README focused on clickhousectl: move Rust settings types, UDF error/model details, and OpenAPI analyzer documentation into crates/clickhouse-cloud-api/README.md, while retaining CLI input/output behavior. Clarify both AGENTS.md files and the completion checklist so library-only changes do not require a root README edit.

Appended after #973 at the top of native stack #876. The branch also includes the documentation corrections already pushed to #897, #940, #944, #945, and #947, so the combined stack head contains the complete cleanup. Relative to #973, only the two READMEs and two AGENTS.md files change.

Audited main and all 58 previously open PRs. The five PRs above were the only ones adding library-only root README content. PR #940 retains its TTL note because the CLI accepts that input.

Validation: formatting and whitespace checks passed after stacking. The final tree exactly matches the previously audited merge simulation of all 56 stack entries plus the cleanup; all existing stack heads are ancestors of this branch. The original main-based cleanup passed default CLI Clippy/tests, telemetry-disabled check and all-target Clippy, library/analyzer all-target Clippy/tests, and 91 Python tests. Stacking introduces no runtime-code changes relative to #973.

Fixes #974.

All 58 direct and cumulative stack merges succeed, but GitHub retains an unknown result and a test-merge ref from older branch heads. Trigger a fresh calculation without changing the source tree.
Carry existing lower-stack README corrections and ancestry forward. The merged tree matches the independently verified cumulative stack result.

@iskakaushik iskakaushik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sdairs
sdairs merged commit a65a9ca into main Sep 17, 2026
9 checks passed
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.

Keep the root README focused on CLI capabilities and behavior

2 participants