Skip to content

Normalize nopre -> QA (+ split AD/Enzyme)#383

Merged
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:normalize-nopre-qa
Jun 15, 2026
Merged

Normalize nopre -> QA (+ split AD/Enzyme)#383
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:normalize-nopre-qa

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor

Part of the fleet-wide nopre/NoPre test-group normalization (policy: standard QA checks live in a canonical QA group on ["lts", "1"], ubuntu-only; pre-incompatible functional tests get their own content-named groups).

Survey result for this repo: the nopre group was PURE standard QA (Aqua + JET), so it merges into a canonical QA group:

  • test/nopre/test/qa/, with aqua_tests.jl + jet_tests.jl folded verbatim into a single canonical qa.jl next to the group Project.toml (content unchanged).
  • test/runtests.jl: dispatches on GROUP == "QA"; error-message group list updated.
  • test/test_groups.toml: [nopre][QA], versions ["lts", "1"] (unchanged), ubuntu-only.

Verification: static only (TOML parse of test_groups.toml + qa/Project.toml; Meta.parseall of changed .jl files; Runic check clean). Test suites not run locally — CI should confirm the QA group goes green.

Ignore until reviewed by @ChrisRackauckas.

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor Author

Branch reset to a clean implementation off the current default branch: removed unrelated changes from the prior revision (no spurious action/codecov version edits), renamed the dep env test/nopre -> test/qa, and kept the existing JET/Aqua test files in place (no test-logic changes). Static-verified locally: test_groups.toml parses and runtests.jl passes Meta.parseall.

The nopre group carried only standard QA checks (JET, Aqua), so it is renamed
to the canonical QA group at versions ["lts","1"]. The dep env test/nopre is
renamed to test/qa and the runtests dispatch keys on GROUP == "QA".

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review June 15, 2026 10:53
@ChrisRackauckas ChrisRackauckas merged commit 4e02334 into SciML:main Jun 15, 2026
6 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