Migrate CI to SciML centralized reusable workflows#95
Merged
ChrisRackauckas merged 1 commit intoJun 2, 2026
Merged
Conversation
Convert Tests, Downgrade, and Runic format-check to the centralized SciML/.github reusable workflows (pinned @v1), add a centralized SpellCheck workflow, and a minimal _typos.toml. - CI.yml: tests.yml@v1 caller (version "1", coverage default true) - Downgrade.yml: downgrade.yml@v1 caller (julia 1.10, skip Pkg,TOML); also fix on: branches master -> main so it actually triggers - FormatCheck.yml: runic.yml@v1 caller (repo already Runic-formatted) - SpellCheck.yml: spellcheck.yml@v1 caller (new) - _typos.toml: exclude Manifest.toml (Missings package false positive) - Fix two real comment typos (Curent->Current, Bechmarking->Benchmarking) GPU.yml (self-hosted CUDA) and TagBot.yml left unchanged. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
2 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.
Normalizes CI to the SciML centralized reusable workflows (
SciML/.github, all callers pinned@v1).Workflows converted
tests.yml@v1caller. Preserves the original matrix (Julia version"1", ubuntu, x64) and coverage collection (codecov viasecrets: inherit).downgrade.yml@v1caller. Preservesjulia-version: 1.10,skip: Pkg,TOML,alldeps/allow-reresolve: falsesemantics. Also fixes a latent bug: the originalon:triggered only on branchmaster, but the default branch ismain, so the Downgrade job never actually ran. Changed tomainso it now triggers.runic.yml@v1caller. The repo already ran Runic (fredrikekre/runic-action) and is already Runic-formatted, so no reformatting was applied (Runic --checkpasses clean locally).Workflows added
spellcheck.yml@v1caller (new; SciML standard).Left unchanged
Pkg.add(CUDA)pin step and Manifest removal) — not part of the centralized CPU standard set and not faithfully expressible through the centralized callers, so preserved as-is.Typos / SpellCheck
crate-ci/typos(v1.47.0) locally reported:Missings(14x, all inbenchmarks/*/Manifest.toml) — false positive (real Julia package name). Handled by a minimal_typos.tomlexcludingManifest.toml.Curent->Current(comment intest/runtests.jl) — real typo, fixed.Bechmarking->Benchmarking(commented-out plot title inbenchmarks/CPU_vs_GPU/benchmark.jl) — real typo, fixed.After these changes
typosexits clean locally.Dependabot / CompatHelper
CompatHelper.ymlexisted; nothing to remove.dependabot.ymlalready conforms to the standard (github-actionsweekly at/;juliadaily at/and/testwithgroups.all-julia-packages.patterns: ["*"], nocrate-ci/typosignore block). The only dirs with aProject.tomlare/and/test, both already listed. No changes needed.Note on branch protection
Check names change with this migration (e.g. the test job now appears under the reusable workflow's job names). Branch-protection required-status-checks must be updated accordingly, or merges may block on stale check names.
🤖 Generated with Claude Code