Skip to content

Clarify that IO documentation is specification-only - #3677

Merged
spmallette merged 1 commit into
apache:masterfrom
spmallette:tp-docs-tinkerpop-docs-voice-update--141
Sep 24, 2026
Merged

spmallette merged 1 commit into
apache:masterfrom
spmallette:tp-docs-tinkerpop-docs-voice-update--141

Conversation

@spmallette

Copy link
Copy Markdown
Contributor

The IO book documents the serialization formats for providers and advanced users implementing against them. Make explicit in books-and-voice.md that these files specify the wire and file format rather than teach how to read or write it in code, so examples that invoke a reader or writer belong in the reference and tutorial books instead. The existing guidance that format-illustrative snippets are appropriate is preserved.

The IO book documents the serialization formats for providers and advanced
users implementing against them. Make explicit in books-and-voice.md that
these files specify the wire and file format rather than teach how to read or
write it in code, so examples that invoke a reader or writer belong in the
reference and tutorial books instead. The existing guidance that
format-illustrative snippets are appropriate is preserved.

Assisted-by: Kiro:claude-opus-4.8
@spmallette
spmallette merged commit ae09b59 into apache:master Sep 24, 2026
6 checks passed
@spmallette
spmallette deleted the tp-docs-tinkerpop-docs-voice-update--141 branch September 24, 2026 21:10
@codecov-commenter

codecov-commenter commented Sep 24, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.70%. Comparing base (a28cd1f) to head (6be6711).
⚠️ Report is 645 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3677      +/-   ##
============================================
+ Coverage     76.35%   76.70%   +0.34%     
- Complexity    13424    14346     +922     
============================================
  Files          1012     1037      +25     
  Lines         60341    64761    +4420     
  Branches       7075     7695     +620     
============================================
+ Hits          46076    49677    +3601     
- Misses        11548    11989     +441     
- Partials       2717     3095     +378     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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