Skip to content

qa: rely on SciMLTesting for rendered API coverage - #4973

Merged
ChrisRackauckas merged 2 commits into
SciML:masterfrom
ChrisRackauckas-Claude:docs/remove-redundant-api-qa
Aug 15, 2026
Merged

qa: rely on SciMLTesting for rendered API coverage#4973
ChrisRackauckas merged 2 commits into
SciML:masterfrom
ChrisRackauckas-Claude:docs/remove-redundant-api-qa

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Note

Please ignore this draft until it has been reviewed by @ChrisRackauckas.

What changed

  • Remove MTK's repository-specific canonical_docs_entries rendered-API test.
  • Remove the deprecated target_defined_modules JET override from run_qa.
  • Require SciMLTesting 2.9, whose scoped rendered-doc check covers each @autodocs block directly.

The manual now relies on the standard SciMLTesting public API and rendered-document checks. The explicit reexport allowlist and genuine ExplicitImports/Aqua ownership exceptions remain unchanged.

This draft depends on SciMLTesting #51 being merged and registered as 2.9.0; its QA environment cannot resolve 2.9 before that release. It is intentionally separate from the JET scalability investigation documented in MTK issue #4958.

Verification

  • Runic check on test/qa/qa.jl: passed.
  • Typos on the changed QA files: passed.
  • TOML parse of test/qa/Project.toml: passed.
  • git diff --check: passed.

The full QA suite is deferred until SciMLTesting 2.9.0 is registered; running the old 2.8 harness here would falsely validate the removed bespoke rendered-doc coverage.

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 August 15, 2026 08:52
@ChrisRackauckas
ChrisRackauckas merged commit 76e76ea into SciML:master Aug 15, 2026
55 of 68 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