Skip to content

Bump test dependency compat bounds (supersedes #6 and #8) - #10

Merged
cossio merged 3 commits into
masterfrom
claude/close-dependabot-prs-nw6027
Aug 26, 2026
Merged

cossio merged 3 commits into
masterfrom
claude/close-dependabot-prs-nw6027

Conversation

@cossio

@cossio cossio commented Aug 25, 2026 •

Copy link
Copy Markdown
Owner

Consolidates the two open dependabot PRs (#6 and #8) into a single change to test/Project.toml, keeping it compliant with TestCompatHygiene.jl's invariant: no [compat] entry in test/Project.toml for a name the root Project.toml already owns.

Compat updates taken from the dependabot PRs (test-only deps):

Deliberately not taken from #6: SpecialFunctions and Statistics — both are deps of the root Project.toml, which already bounds them. Workspace members share one manifest, so repeating those bounds in test/Project.toml would be intersected with the root's and silently narrow the real resolve (the exact failure mode TestCompatHygiene guards against).

Validated locally on Julia 1.12.1: the workspace resolves, the full test suite passes (Aqua included), and TestCompatHygiene.check_test_compat reports no offenders.

🤖 Generated with Claude Code

https://claude.ai/code/session_013Po9AaxYCxkBVex5BZAYj1

Applies the test/Project.toml compat updates from the all-julia-packages
group: Aqua 0.6, 0.8; Distributions 0.25.131; FiniteDifferences 0.12.34;
SafeTestsets 0.1.0; SpecialFunctions 2.9.0; Statistics 1.11.1;
Zygote 0.7.12.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013Po9AaxYCxkBVex5BZAYj1
SpecialFunctions and Statistics are deps of the root Project.toml, which
already declares compat for them. Since [workspace] projects share one
manifest, repeating a bound in test/Project.toml would be intersected
with the root's and silently narrow the real resolve. Keep compat here
for test-only deps exclusively, per TestCompatHygiene.jl's invariant.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013Po9AaxYCxkBVex5BZAYj1
@cossio
cossio enabled auto-merge August 26, 2026 10:22
Comment thread test/Project.toml Outdated
Requested in review.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013Po9AaxYCxkBVex5BZAYj1
@cossio
cossio merged commit 953a749 into master Aug 26, 2026
2 checks passed
@cossio
cossio deleted the claude/close-dependabot-prs-nw6027 branch October 11, 2026 08:53
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