Skip to content

Document public API coverage - #1255

Merged
ChrisRackauckas merged 8 commits into
SciML:masterfrom
ChrisRackauckas-Claude:agent/public-doc-audit
Jul 12, 2026
Merged

Document public API coverage#1255
ChrisRackauckas merged 8 commits into
SciML:masterfrom
ChrisRackauckas-Claude:agent/public-doc-audit

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Ignore until reviewed by @ChrisRackauckas.

Summary:

  • Add source docstrings for public/exported Optimization API names across backend packages.
  • Add rendered docs entries and package docs pages where coverage was missing.
  • Add QA coverage that statically checks exported/public names for source docstrings and @docs entries.
  • Replace the docs ColPrac README link with the canonical ColPrac page to avoid GitHub linkcheck 429s.

Verification:

  • timeout 3600 julia --startup-file=no -m Runic -i
  • timeout 3600 env JULIA_DEPOT_PATH=... julia --project=test/qa test/qa/qa.jl
  • timeout 3600 env JULIA_DEPOT_PATH=... julia --project=docs docs/make.jl

Add source docstrings and rendered docs entries for public Optimization API names across backend packages. Add QA coverage for public/exported API docstrings and docs entries.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Remove the copied public API docs scanner from Optimization QA and configure SciMLTesting's built-in api_docs check with explicit ignores for external reexports that this repo does not own.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Add gradient_sampling_subsolver! to the OptimizationManopt API docs QA ignore list because it is reexported from Manopt and is not owned by OptimizationManopt.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
SciMLTesting develops the package under test before running the QA group, so the QA project does not need a self `[sources]` path entry. Removing it avoids leaking local checkout paths into the PR.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review July 12, 2026 07:21
@ChrisRackauckas
ChrisRackauckas merged commit 0c663b6 into SciML:master Jul 12, 2026
146 of 152 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.

2 participants