Skip to content

docs: reword the README intro and API pages - #72

Merged
Reddimus merged 1 commit into
mainfrom
docs/readme-wording
Sep 26, 2026
Merged

Reddimus merged 1 commit into
mainfrom
docs/readme-wording

Conversation

@Reddimus

@Reddimus Reddimus commented Sep 26, 2026 •

Copy link
Copy Markdown
Owner

Rewords the user-facing docs to say where the library's types come from.

  • README intro: "A C++23 client for Kalshi's Predictions API... Its types and methods come from Kalshi's OpenAPI and AsyncAPI documents in spec/, so their names match Kalshi's API docs." The contributing pointer now says "updating the specs".
  • docs/operations.md and docs/channels.md now open with "Covers Kalshi's ... document". The change is in their templates in tools/codegen/, followed by make codegen, which changed only those two lines.
  • include/README.md drops the per-row build labels in the header table, and the kalshi::VERSION brief now says it comes from project(... VERSION).

Left alone on purpose: the "Do not edit" banners at the top of the code generator's headers, the contributor docs that explain make codegen, and past CHANGELOG entries. There's no CHANGELOG entry, since this is wording only.

Testing

make lint (including the codegen check), make test (287 passed), ./tools/test_consumers.sh, make docs, and markdownlint.

The README, the operations and channels pages, and the public header
table now say where the types come from instead of how they are made.
Copilot AI lite review requested due to automatic review settings September 26, 2026 18:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Reddimus
Reddimus merged commit 4138820 into main Sep 26, 2026
14 checks passed
@Reddimus
Reddimus deleted the docs/readme-wording branch September 26, 2026 18:37
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.

2 participants