Skip to content

docs: add @example blocks and flesh out sparse JSDoc#19

Merged
mjradwin merged 1 commit into
mainfrom
docs/jsdoc-examples
May 25, 2026
Merged

docs: add @example blocks and flesh out sparse JSDoc#19
mjradwin merged 1 commit into
mainfrom
docs/jsdoc-examples

Conversation

@mjradwin

Copy link
Copy Markdown
Member

Summary

  • Adds @example blocks to public functions and methods across greg, hdateBase, dateFormat, pad, gematriya, locale, hebrewStripNikkud, and HDate.add so the TypeDoc reference at https://hebcal.github.io/api/hdate/ shows runnable usage for each entry point.
  • Fills in JSDoc descriptions that were missing or unhelpful:
    • getYahrzeitHD and getBirthdayHD had no JSDoc at all
    • hd2abs description was a verbatim copy of hebrew2abs; now explains it as the SimpleHebrewDate wrapper
    • AnniversaryDate type now documents its three accepted forms
    • isoDateString, Locale.addTranslations, and Locale.copyLocaleNoNikud expanded beyond their one-line stubs

No runtime behavior changes — comment-only edits.

Test plan

  • `npm test` passes (105 passed / 3 skipped)
  • `npm run build` succeeds
  • Spot-check rendered TypeDoc output after merge

🤖 Generated with Claude Code

Adds @example to public functions and methods that lacked them across
greg, hdateBase, dateFormat, pad, gematriya, locale, hebrewStripNikkud,
and HDate.add so the generated TypeDoc reference is self-explanatory.

Also fills in descriptions that were missing or unhelpful:
- getYahrzeitHD and getBirthdayHD had no JSDoc at all
- hd2abs description was a copy of hebrew2abs; now explains the wrapper
- AnniversaryDate type now documents its three accepted forms
- isoDateString, Locale.addTranslations, and Locale.copyLocaleNoNikud
  expanded beyond their one-line stubs

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mjradwin mjradwin merged commit 1bebf82 into main May 25, 2026
4 checks passed
@mjradwin mjradwin deleted the docs/jsdoc-examples branch May 25, 2026 23:01
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