Skip to content

ci: standardize linting and CI workflows, format with Runic - #23

Open
adrhill wants to merge 8 commits into
mainfrom
ah/ci-cleanup
Open

ci: standardize linting and CI workflows, format with Runic#23
adrhill wants to merge 8 commits into
mainfrom
ah/ci-cleanup

Conversation

@adrhill

@adrhill adrhill commented Jul 13, 2026

Copy link
Copy Markdown
Member
  • Format the package with Runic, drop .JuliaFormatter.toml, and add a Runic CI job.
  • Move the Aqua.jl and JET.jl checks into a self-contained test/linting.jl (JET runs on Julia v1.12+ only).
  • Add ExplicitImports.jl import-hygiene checks and a package-scoped Aqua.test_ambiguities check.
  • Standardize the GitHub Actions workflow (pin action majors, Julia matrix 1/lts, single ubuntu-latest runner) and add a pre-commit config.

🤖 Generated with Claude Code

- Format the package with Runic: add a Runic CI job, remove `.JuliaFormatter.toml`, and drop the JuliaFormatter format test and test dependency.
- Move the Aqua.jl and JET.jl checks into a self-contained `test/linting.jl`; JET runs on Julia v1.12+ only.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@adrhill adrhill changed the title Migrate formatting to Runic, standardize linting ci: format with Runic, standardize Aqua and JET linting Jul 13, 2026
adrhill and others added 6 commits July 13, 2026 21:27
Add `Aqua.test_ambiguities(<Module>)` alongside the existing
`Aqua.test_all(...; ambiguities = false)`. test_all's built-in check
broadens to [Module, Core, Base] and trips on dependency-owned
ambiguities we can't fix; the scoped call only reports ambiguities where
at least one conflicting method is owned by the package itself.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Pin all GitHub Actions to their latest major moving tag: checkout@v7,
setup-julia@v3, cache@v3, codecov-action@v7 (julia-actions/*, TagBot and
runic-action stay @v1). The codecov bump is input-safe as every step
already uses the post-v4 `files:` schema.

Test only on Julia '1' and 'lts' on a single ubuntu-latest runner: drop
'pre'/'nightly'/pinned point releases and remove the os/arch matrix
dimensions and hardcoded arch inputs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
# Conflicts:
#	.github/workflows/CI.yml
#	.pre-commit-config.yaml
#	test/runtests.jl
@adrhill adrhill changed the title ci: format with Runic, standardize Aqua and JET linting ci: standardize linting and CI workflows, format with Runic Jul 13, 2026
Co-Authored-By: Claude Opus 4.8 (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