WHAT: CI.yml tests 1.10/1.11/'1' across three OSes but never runs julia-processcoverage + codecov (no coverage signal), has no nightly/'pre' entry (valuable for a package leaning on LinearAlgebra/SparseArrays internals like getcolptr/rowvals), and there is TagBot.yml but no CompatHelper.yml (so [compat] bounds, including the new deps, won't auto-bump). Separately, the README quotes many specific perf numbers (QR ms tables, lstsq '191x at n=2000', '44x reusable solve') with no benchmark/ dir, no BenchmarkTools dep, and no PkgBenchmark/AirspeedVelocity — nothing guards the new QR/lstsq/refactor paths against regression, and the allocation drifts found in this review are evidence the numbers have already drifted. WHY IT MATTERS: No regression safety net or coverage signal on the newest, most fragile surface. FIX: Add coverage upload to CI.yml, a non-blocking nightly matrix entry (fail-fast:false + continue-on-error), a CompatHelper.yml, and a benchmark/benchmarks.jl (BenchmarkTools SUITE) covering Woodbury/augmented/QR factor-solve-refactor, update_lowrank!, lstsq structured-vs-dense, and the cached-lstsq reuse path. EFFORT: M.
Priority: low. Filed from an automated next-steps audit of the QR/lstsq work (see PR #6).
WHAT: CI.yml tests 1.10/1.11/'1' across three OSes but never runs julia-processcoverage + codecov (no coverage signal), has no nightly/'pre' entry (valuable for a package leaning on LinearAlgebra/SparseArrays internals like getcolptr/rowvals), and there is TagBot.yml but no CompatHelper.yml (so [compat] bounds, including the new deps, won't auto-bump). Separately, the README quotes many specific perf numbers (QR ms tables, lstsq '191x at n=2000', '44x reusable solve') with no benchmark/ dir, no BenchmarkTools dep, and no PkgBenchmark/AirspeedVelocity — nothing guards the new QR/lstsq/refactor paths against regression, and the allocation drifts found in this review are evidence the numbers have already drifted. WHY IT MATTERS: No regression safety net or coverage signal on the newest, most fragile surface. FIX: Add coverage upload to CI.yml, a non-blocking nightly matrix entry (fail-fast:false + continue-on-error), a CompatHelper.yml, and a benchmark/benchmarks.jl (BenchmarkTools SUITE) covering Woodbury/augmented/QR factor-solve-refactor, update_lowrank!, lstsq structured-vs-dense, and the cached-lstsq reuse path. EFFORT: M.
Priority: low. Filed from an automated next-steps audit of the QR/lstsq work (see PR #6).