Skip to content

fix: move cancellation docs into docs/ so they survive README regeneration#430

Merged
doron050 merged 1 commit into
mainfrom
fix/cancellation-docs-in-readme
Jul 25, 2026
Merged

fix: move cancellation docs into docs/ so they survive README regeneration#430
doron050 merged 1 commit into
mainfrom
fix/cancellation-docs-in-readme

Conversation

@doron050

Copy link
Copy Markdown
Collaborator

Problem

PR #420 added the withCancellationToken option and cancellation documentation directly to README.md. However, README.md is auto-generated by generate_all_docs.sh which concatenates all docs/0*.md files. When the release prep workflow (gen-release-pr.yml) regenerates the README, the cancellation content gets lost — as seen in the auto-generated release prep PR #428.

Fix

  1. Added the withCancellationToken row to the options table in docs/03_Usage.md
  2. Added the full ### Cancellation section (with code example and notes) to docs/03_Usage.md after the Transactions section
  3. Regenerated README.md from the docs via generate_all_docs.sh

Now when the release prep workflow runs, the cancellation docs will be preserved in the regenerated README.

Verification

Ran ./docs/scripts/generate_all_docs.sh locally and verified the cancellation content appears in the regenerated README at the expected location (between Transactions and PostgreSQL sections).

…ADME regeneration

The cancellation documentation (withCancellationToken option and Cancellation
section) was added directly to README.md in PR #420. However, README.md is
auto-generated by concatenating docs/0*.md files via generate_all_docs.sh.
When the release prep workflow regenerates the README, the cancellation
content gets lost because it lives only in README.md itself.

Fix: move the content into docs/03_Usage.md (the Usage doc where the options
table and transactions section live), then regenerate README.md from the docs.
@github-actions

Copy link
Copy Markdown
Contributor

Release Assistant - Passed

No version-worthy changes detected. Only non-functional, doc, or test changes.

@doron050
doron050 merged commit 6499625 into main Jul 25, 2026
3 checks passed
This was referenced Jul 25, 2026
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