Skip to content

docs: document every method with YARD and add a docs rake task - #22

Merged
tas50 merged 1 commit into
mainfrom
yard-docs
Aug 28, 2026
Merged

tas50 merged 1 commit into
mainfrom
yard-docs

Conversation

@tas50

@tas50 tas50 commented Aug 28, 2026

Copy link
Copy Markdown
Member

docs: document every method with YARD and add a docs rake task

Nothing in this repository carried API documentation beyond a stray author tag.
Every method now has a YARD comment covering what it does, what it takes, what
it returns, and -- where it matters -- why it is written the way it is. yard stats --private reports 100% documented.

rake doc generates the HTML into doc/, which is already gitignored.

yard sits in the :development bundler group, which CI omits when it runs the
tests, so the Rakefile only defines the real task when yard is installed and
otherwise leaves a stub that says how to get it. Nothing in CI gates on
documentation
, and rake test still works with the group omitted -- verified
by running it under BUNDLE_WITHOUT=development, the same setting the shared
workflow uses.

Nothing in this repository carried API documentation beyond a stray author tag.
Every method now has a YARD comment covering what it does, what it takes, what
it returns, and -- where it matters -- why it is written the way it is. `yard
stats --private` reports 100% documented.

`rake doc` generates the HTML into `doc/`, which is already gitignored.

yard sits in the `:development` bundler group, which CI omits when it runs the
tests, so the Rakefile only defines the real task when yard is installed and
otherwise leaves a stub that says how to get it. **Nothing in CI gates on
documentation**, and `rake test` still works with the group omitted -- verified
by running it under `BUNDLE_WITHOUT=development`, the same setting the shared
workflow uses.

Signed-off-by: Tim Smith <tsmith84@proton.me>
@tas50
tas50 merged commit 1c76250 into main Aug 28, 2026
8 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.

1 participant