Skip to content

Export list and docs coverage - #145

Open
alecloudenback wants to merge 1 commit into
refactor/09-select-rowfrom
refactor/10-exports-docs
Open

alecloudenback wants to merge 1 commit into
refactor/09-select-rowfrom
refactor/10-exports-docs

Conversation

@alecloudenback

Copy link
Copy Markdown
Member

MakehamGompertz was exported but never defined (MortalityTables.MakehamGompertz is an UndefVarError). The three Dukes-MacDonald functions were on no docs page.

Changes

  • MakehamGompertz removed from the export list. The other laws stay unexported, as the parametric docs page explains.
  • New "Select-period deterioration (Dukes-MacDonald)" section on the Tables page with a @docs block for the three functions (still unexported).
  • The hazard/cumhazard/μ docs block on the parametric page is canonical=false like every other page: with the docstrings now on bare function declarations (PR 6) it collided with the reference page's @autodocs. master builds cleanly; this branch did not until this change.

Testing

  • New test that every name in names(MortalityTables) is defined.
  • julia --project=docs docs/make.jl builds with no missing-docstring warnings.

Stacked on #144 (base branch refactor/09-select-row). Merge the stack in order; once the base PR is merged and its branch deleted, GitHub retargets this PR to master.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT

MakehamGompertz was exported but never defined. The three
dukes_macdonald functions now have a section on the Tables docs page
(still unexported). The hazard/cumhazard/μ docs block on the
parametric page is marked non-canonical like every other page, since
the reference page's autodocs is canonical; with the docstrings now on
bare function declarations the two otherwise collide.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JFnc6pvofuNXTW2xdCcewT

This branch has not been deployed

No deployments
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