Skip to content

ci: add Trim test group to CI matrix#3328

Open
asinghvi17 wants to merge 4 commits intoSciML:masterfrom
asinghvi17:as/trim-ci
Open

ci: add Trim test group to CI matrix#3328
asinghvi17 wants to merge 4 commits intoSciML:masterfrom
asinghvi17:as/trim-ci

Conversation

@asinghvi17
Copy link
Copy Markdown
Contributor

Summary

  • Adds Trim to the CI matrix, running JuliaC --trim=safe compilation tests
  • Excluded on lts and 1 since trim requires Julia 1.12+
  • Depends on the trim test infrastructure from the trimmability PR

Test plan

  • Verify the Trim job runs on pre only
  • Confirm it picks up GROUP=Trim in runtests.jl

🤖 Generated with Claude Code

asinghvi17 and others added 3 commits April 2, 2026 09:59
Runs the trim (JuliaC --trim=safe) compilation tests on Julia pre-release
(1.12+). Excludes lts and 1 since trim requires Julia 1.12+.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add test/trim/ sub-environment following the existing pattern
(like test/ad/, test/downstream/):
- trim_tests.jl generates per-solver test projects (main.jl +
  Project.toml) from a SOLVER_CONFIGS dict, compiles each with
  JuliaC --trim=safe, and validates the output.
- Wire into runtests.jl as GROUP="Trim" (requires Julia 1.12+).
- Tests: tsit5, fbdf, rodas5p, auto (DefaultODEAlgorithm).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Capture the time type directly from `t::_tType` instead of passing
`eltype(tType)` as a separate argument, per review feedback from
@oscardssmith.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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