Skip to content

CI: add SpellCheck workflow and clean up Dependabot config (#371)#373

Merged
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:ci/sciml-standard-spellcheck-dependabot
May 30, 2026
Merged

CI: add SpellCheck workflow and clean up Dependabot config (#371)#373
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:ci/sciml-standard-spellcheck-dependabot

Conversation

@ChrisRackauckas-Claude
Copy link
Copy Markdown
Contributor

Note

Please ignore until reviewed by @ChrisRackauckas. Opened as a draft.

Closes #371, and brings the CI in line with the SciML standard (using Sundials.jl as the reference).

What this changes

.github/dependabot.yml

  • Removed enable-beta-ecosystems: true. The current Dependabot options reference lists this key as "Not currently in use" — the Julia ecosystem no longer requires it, so it's dead config (issue point 1).
  • Moved the crate-ci/typos ignore rule out of the julia ecosystem and into the github-actions ecosystem. crate-ci/typos is a GitHub Action, not a Julia package, so the ignore had no effect where it was (issue point 2). It belongs alongside the action so the version pinned in the new SpellCheck workflow isn't bumped by patch/minor PRs — this matches Sundials.jl's dependabot.yml.

.github/workflows/SpellCheck.yml (new)

  • Standard SciML spell-check workflow running crate-ci/typos on PRs, matching Sundials.jl/.github/workflows/SpellCheck.yml. This is the thing @ChrisRackauckas asked to "enable here too" in the issue thread. Pinned to v1.34.0.

.typos.toml (new)

  • typos flags the temp_noth test variable in test/core_tests.jl (it wants north). Added a single [default.extend-words] exception for noth rather than renaming a test variable. This is the only false positive in the repo.

Verification

Ran crate-ci/typos v1.34.0 locally against the repo (same version the workflow pins):

  • Before the config: 3 errors (all the temp_noth occurrences).
  • After adding .typos.toml: exit 0, clean.

Both YAML files validated with yaml.safe_load.

Not included here (possible follow-up)

Sundials.jl's Tests.yml/Downgrade.yml use the reusable SciML/.github/.github/workflows/tests.yml@v1 workflow. ComponentArrays.jl still uses a self-contained ci.yml with a group matrix (Core/Autodiff/GPU/Downstream/Reactant/nopre). Converting to the reusable workflow is the larger remaining "match Sundials" step, but the reusable workflow uses codecov-action with fail_ci_if_error: true + a CODECOV_TOKEN secret, which I can't verify is configured for this repo without risking red CI. Happy to do that conversion as a separate PR if wanted.

Co-Authored-By: Chris Rackauckas accounts@chrisrackauckas.com

🤖 Generated with Claude Code

Brings CI in line with the SciML standard (matching Sundials.jl):

- Add a Spell Check workflow using crate-ci/typos, with a .typos.toml
  configuring the one project-specific exception (`temp_noth` test
  variable).
- Move the `crate-ci/typos` Dependabot ignore from the `julia`
  ecosystem (where it had no effect — typos is a GitHub Action) to the
  `github-actions` ecosystem, so the pinned typos version in the new
  workflow is not bumped by patch/minor PRs.
- Drop `enable-beta-ecosystems: true`; per the current Dependabot
  options reference this flag is "Not currently in use", so the Julia
  ecosystem no longer requires it.

Resolves the cleanup requested in SciML#371.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review May 30, 2026 08:26
@ChrisRackauckas ChrisRackauckas merged commit e8d17bf into SciML:main May 30, 2026
19 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.

Update Dependabot Config

2 participants