Centralize CI on SciML reusable workflows#597
Merged
ChrisRackauckas merged 2 commits intoJun 4, 2026
Conversation
Convert all CI to the centralized SciML reusable workflows (pinned @v1, secrets: "inherit" on every caller): - Documentation: inline build/deploy -> documentation.yml@v1 - FormatCheck (format-check.yml@v1, SciMLStyle) -> Runic.yml using runic.yml@v1 - SpellCheck: inline crate-ci/typos -> spellcheck.yml@v1 - Downgrade: inline -> downgrade.yml@v1 (preserving the disabling `if: false`) - Tests already used tests.yml@v1 (unchanged; matrix preserved) - GPU and TagBot left unchanged (no central equivalents) Apply Runic formatting across the package (the repo previously used SciMLStyle via format-check). Fix two genuine typos (Aggegators, simmulation) and add a _typos.toml for domain false positives (McCollum surname, LambaEM solver, resetted_jump_problem, Numer journal abbreviation, setp). Remove the crate-ci/typos dependabot ignore and point the julia dependabot block at the actual Project.toml dirs. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Member
|
This seems to be reformatting the repo? Please revert those commits and keep this to the minimal set of commits for updating the CI. |
JumpProcesses ran the centralized JuliaFormatter format-check, not Runic. Switching it to the centralized Runic check reformatted the entire repo (~85 files), which is too large to bundle with the CI migration. Restore the original FormatCheck.yml (JuliaFormatter) and revert the reformatting; Runic adoption can be a separate, focused PR. Kept: Documentation/Downgrade/SpellCheck central callers, dependabot cleanup, _typos.toml, and the 2 real typo fixes the spell check surfaced. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
isaacsas
approved these changes
Jun 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Please ignore until reviewed by @ChrisRackauckas.
Converts JumpProcesses.jl CI to the centralized SciML reusable workflows. Every caller pins
@v1and passessecrets: "inherit".Converted (inline -> central caller)
documentation.yml@v1(coverage behavior preserved:--code-coverage=user+ processcoverage + codecov).FormatCheck.yml(which usedformat-check.yml@v1, i.e. SciMLStyle) replaced byRunic.ymlusingrunic.yml@v1.crate-ci/typos->spellcheck.yml@v1.downgrade.yml@v1(julia-version1.10, skipPkg,TOML, allow-reresolvefalse); the existingif: falsedisable is preserved.Unchanged
tests.yml@v1withsecrets: "inherit"; the exact version x group matrix (1/lts/prexInterfaceI/InterfaceII/QA, excludingpre+QA) is untouched.Formatting / typos
format-check), so switching to Runic produced formatting changes in mostsrc/test/ext/docs.jlfiles. Runic ran cleanly (no parse errors), so noexclude:was needed.Aggegators->Aggregators(docs),simmulation->simulation(test comment)._typos.tomlfor domain false-positives (not prose edits):McCollumsurname,LambaEMsolver name,resetted_jump_probleminternal function,Numer(journal abbreviation),setp(SymbolicIndexingInterface).typos .now exits 0.Cleanup
crate-ci/typosignore fromdependabot.yml.juliadependabot block at the dirs that actually have aProject.toml(/,/docs,/test/gpu); the old list included/test, which has no manifest.CompatHelper.ymlpresent.Heads up
Check names change (e.g.
Format Check->Runic, inline doc/spellcheck job names -> the reusable workflow job names). Branch-protection required-status-check names will need updating accordingly.🤖 Generated with Claude Code