Skip to content

Describe storage internals in our own terms - #54

Merged
matrixarkai merged 1 commit into
mainfrom
oss/comment-hygiene
Aug 19, 2026
Merged

Describe storage internals in our own terms#54
matrixarkai merged 1 commit into
mainfrom
oss/comment-hygiene

Conversation

@bjmeetsfo

Copy link
Copy Markdown
Collaborator

Comment-only pass across the crate. Descriptions of the storage internals now
stand on their own rather than being framed as comparisons.

No behaviour change: comment text only, no code, identifiers or string literals.

Deliberately untouched:

  • serde alias attributes — on-disk backward compatibility; removing one
    stops existing data from being read.
  • configuration identifiers and environment variable names — renaming those
    is a breaking change for deployments, not a documentation fix.
  • our own type and function names, and ordinary English uses of the word
    reference (dangling reference, live index references).

@bjmeetsfo
bjmeetsfo requested a review from superhaiou as a code owner August 19, 2026 03:12
Comment-only pass across the crate. Descriptions now stand on their own rather
than attributing a design to, or naming symbols from, another codebase.

No behaviour change. Deliberately untouched:
  * serde alias attributes -- on-disk backward compatibility; removing one stops
    existing data from being read;
  * configuration identifiers and environment variable names -- renaming those
    is a breaking change for deployments, not a documentation fix;
  * ordinary English uses of the word reference (dangling reference, live index
    references).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@bjmeetsfo
bjmeetsfo force-pushed the oss/comment-hygiene branch from a839f7b to 054a9ff Compare August 19, 2026 03:14
@matrixarkai
matrixarkai merged commit eef5f50 into main Aug 19, 2026
3 of 5 checks passed
@matrixarkai
matrixarkai deleted the oss/comment-hygiene branch August 19, 2026 05:57
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.

3 participants