Skip to content

ci(downgrade): expose julia-downgrade-compat mode#64

Merged
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:downgrade-mode-input
Jun 5, 2026
Merged

ci(downgrade): expose julia-downgrade-compat mode#64
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:downgrade-mode-input

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown

Adds a mode input to downgrade.yml, forwarded to julia-actions/julia-downgrade-compat: deps (direct), alldeps (deps + weakdeps), weakdeps, or forcedeps (strict lower-bound verification). Empty = the action's default, so existing callers are unchanged. Useful for the downgrade re-enablement work in #56 (e.g. forcedeps to verify lower bounds are real).

actionlint clean.

Ignore until reviewed by @ChrisRackauckas.

🤖 Generated with Claude Code

Add a `mode` input (deps/alldeps/weakdeps/forcedeps) forwarded to
julia-actions/julia-downgrade-compat. forcedeps does strict lower-bound
verification; alldeps/weakdeps cover weak dependencies. Empty = action default,
preserving current behavior.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review June 5, 2026 09:53
@ChrisRackauckas ChrisRackauckas merged commit d5cbdc6 into SciML:master Jun 5, 2026
1 of 2 checks passed
ChrisRackauckas added a commit that referenced this pull request Jun 6, 2026
PR #64 added a `mode` input defaulting to "" and passed it through
verbatim as `mode: "${{ inputs.mode }}"`. julia-downgrade-compat rejects
an empty mode (`mode in valid_modes || error()`, downgrade.jl), so every
downgrade.yml@v1 caller that does not set `mode` explicitly errors at the
downgrade-compat step before tests run — breaking downgrade CI org-wide.

PR #67 added the `project` input but did NOT include this fix, so the
regression is still live on @v1. Default `mode` to "deps" and map an
explicit empty string to "deps" as well.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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