Commit d19e184
fix: correct root test env — drop ShorthandConstructors [sources] and downstream-only NLsolve
The branch had added RecursiveArrayToolsShorthandConstructors to the umbrella
root [sources]. It is a test-only dependency (the VA[]/AP[] shorthand tests),
not a dependency of the RecursiveArrayTools package itself, so sourcing it at
the package level makes Pkg.test add it to the Core test env as a declared
package dependency. Aqua.test_stale_deps then flags it as a stale dep of
RecursiveArrayTools (the module never imports it), failing the QA group.
master never had this [sources] entry: its runtests.jl Pkg.develops the
sublibrary into the test env directly, which the current runtests.jl still
does. Removing the [sources] entry restores master's green QA behavior
(verified: Core+QA Pkg.test now passes — Quality Assurance 9 pass / 1 broken
(expected ambiguities @test_broken) / 0 fail; previously 1 fail).
Also drop NLsolve from the main [extras]/[targets].test/[compat]: it is used
only by test/downstream/odesolve.jl and is already declared in
test/downstream/Project.toml, so it does not belong in the base test env.
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>1 parent d592b77 commit d19e184
1 file changed
Lines changed: 1 addition & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | 51 | | |
55 | 52 | | |
56 | 53 | | |
| |||
64 | 61 | | |
65 | 62 | | |
66 | 63 | | |
67 | | - | |
68 | 64 | | |
69 | 65 | | |
70 | 66 | | |
| |||
93 | 89 | | |
94 | 90 | | |
95 | 91 | | |
96 | | - | |
97 | 92 | | |
98 | 93 | | |
99 | 94 | | |
| |||
109 | 104 | | |
110 | 105 | | |
111 | 106 | | |
112 | | - | |
| 107 | + | |
0 commit comments