From f6c476f33fb046837465493c2700d19bfd2af165 Mon Sep 17 00:00:00 2001 From: ChrisRackauckas-Claude Date: Sat, 6 Jun 2026 13:17:55 -0400 Subject: [PATCH] ci(downgrade): default mode to "deps" (fix empty-mode regression) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Co-Authored-By: Claude Opus 4.8 (1M context) --- .github/workflows/downgrade.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/downgrade.yml b/.github/workflows/downgrade.yml index bd63fe1..7fd80f2 100644 --- a/.github/workflows/downgrade.yml +++ b/.github/workflows/downgrade.yml @@ -29,8 +29,8 @@ on: required: false type: string mode: - description: "julia-downgrade-compat mode: deps (direct), alldeps (deps+weakdeps), weakdeps, or forcedeps (strict lower-bound verification). Empty uses the action default." - default: "" + description: "julia-downgrade-compat mode: deps (direct), alldeps (deps+weakdeps), weakdeps, or forcedeps (strict lower-bound verification). Empty maps to 'deps' (the action rejects an empty mode)." + default: "deps" required: false type: string allow-reresolve: @@ -64,7 +64,7 @@ jobs: with: skip: "${{ inputs.skip }}" projects: "${{ inputs.projects }}" - mode: "${{ inputs.mode }}" + mode: "${{ inputs.mode != '' && inputs.mode || 'deps' }}" - uses: julia-actions/julia-buildpkg@v1 with: