Skip to content

Require Aqua 0.8.14+ so downgrade CI works on Julia 1.12 - #17

Merged
cossio merged 1 commit into
mainfrom
claude/fix-ci-wchx4o
Aug 24, 2026
Merged

Require Aqua 0.8.14+ so downgrade CI works on Julia 1.12#17
cossio merged 1 commit into
mainfrom
claude/fix-ci-wchx4o

Conversation

@cossio

@cossio cossio commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Problem

The CI Downgrade compat job has been red on main since the switch to the [workspace] layout (runs 82–86). The downgrade action (v2.7, Resolver.jl --min=@deps) now computes minimal versions for the whole workspace, so the test suite runs with Aqua v0.8.0 — the exact lower bound of the previous Aqua = "0.8" compat. Aqua 0.8.0 predates Julia 1.12 and breaks on it:

  • the Piracy check errors with FieldError: type Core.TypeName has no field mt (the field was removed by Julia 1.12's method-table changes), and
  • the persistent-tasks check fails with "done.log was not created, but precompilation exited".

The regular test job passes with Aqua v0.8.16 on the same Julia 1.12.7, confirming the lower bound is the sole culprit.

Fix

Raise the Aqua lower bound in test/Project.toml to 0.8.14, the earliest release adapted to the Julia 1.12 method-table changes per Aqua's changelog.

Verification

Reproduced the downgrade job locally on Julia 1.12.7 with all deps pinned to the versions the CI run resolved (LogExpFunctions 0.3.26, ChainRulesCore/ChainRulesTestUtils 1.0.0, JLArrays 0.3.0, StaticArrays 1.5.17, ExplicitImports 1.15.0) and Aqua at 0.8.14: the full suite passes, including all 11 aqua tests.

🤖 Generated with Claude Code

https://claude.ai/code/session_012KLPsCxbd3nAyryV2QETpM


Generated by Claude Code

The downgrade job resolves minimal versions for the whole workspace,
so it tests with Aqua 0.8.0, which predates Julia 1.12: its Piracy
check reads the removed Core.TypeName.mt field and errors, and its
persistent-tasks check fails. Aqua 0.8.14 is the earliest release
adapted to the 1.12 method-table changes; the full test suite passes
with it under the downgraded-deps resolution on Julia 1.12.7.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012KLPsCxbd3nAyryV2QETpM
@cossio
cossio enabled auto-merge August 24, 2026 22:10
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (a2923b8) to head (ca953cd).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           74        74           
=========================================
  Hits            74        74           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cossio
cossio merged commit 7f56a17 into main Aug 24, 2026
10 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.

2 participants