Skip to content

Fix workflow input: rename build-mode to build_mode#16

Open
sclaiborne wants to merge 1 commit into
mainfrom
fix/build-mode-input
Open

Fix workflow input: rename build-mode to build_mode#16
sclaiborne wants to merge 1 commit into
mainfrom
fix/build-mode-input

Conversation

@sclaiborne
Copy link
Copy Markdown
Member

What

Renames the workflow_dispatch input build-mode to build_mode and updates the dot-notation reference accordingly.

Why

Hyphenated input names get parsed as subtraction in ${{ inputs. }} expressions — inputs.build-mode is interpreted as inputs.build minus mode, which always evaluates to null. Combined with the || 'Build' fallback, manual workflow_dispatch runs effectively ignored the user's selection and always built as 'Build'. PR triggers were unaffected.

Snake_case lets the dot-notation reference work correctly. The || 'Build' fallback continues to handle the missing-inputs context on pull_request triggers.

Same fix landed across the AS6 sweep so all repos converge on the same template.

Caught by Copilot review on a sibling repo's workflow.

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.

1 participant