Skip to content

Restore project metadata before downgrade QA - #127

Merged
ChrisRackauckas merged 3 commits into
SciML:masterfrom
ChrisRackauckas-Claude:codex/qa-project-metadata
Aug 17, 2026
Merged

Restore project metadata before downgrade QA#127
ChrisRackauckas merged 3 commits into
SciML:masterfrom
ChrisRackauckas-Claude:codex/qa-project-metadata

Conversation

@ChrisRackauckas-Claude

@ChrisRackauckas-Claude ChrisRackauckas-Claude commented Aug 17, 2026

Copy link
Copy Markdown
Member

DowngradeCI promotes old-style [targets].test dependencies into [deps], so Aqua sees temporary test-only packages as stale runtime dependencies during QA. For QA jobs, save every project listed in projects before downgrade and restore its original Project.toml after the downgraded build; the minimum manifest remains in place, so functional groups still test the floor-resolved environment while QA inspects the package’s real dependency tables.

When a root test uses a secondary project such as test, combined downgrade resolution also writes a fixed manifest there. The workflow now moves those secondary manifests aside before root Pkg.test, preventing Julia from trying to merge two fixed environments.

Verification:

  • git diff --check, actionlint, and typos .github/workflows/downgrade.yml passed.
  • The temporary .,test save/restore and secondary-manifest simulation passed.
  • Local LHLFactorization QA with restored project metadata passed: 19/19 tests.

No public API changes and no deprecation path is applicable. Please ignore this PR until it has been reviewed by @ChrisRackauckas.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review August 17, 2026 17:27
@ChrisRackauckas
ChrisRackauckas merged commit a646ba5 into SciML:master Aug 17, 2026
3 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