Skip to content

Fix YAML syntax errors in patch-installer-cd.yml workflow#461

Merged
jasonleenaylor merged 2 commits into
release/9.3from
copilot/fix-b3f109f8-31e3-41b2-ac2d-c9b213a3522f
Sep 16, 2025
Merged

Fix YAML syntax errors in patch-installer-cd.yml workflow#461
jasonleenaylor merged 2 commits into
release/9.3from
copilot/fix-b3f109f8-31e3-41b2-ac2d-c9b213a3522f

Conversation

Copilot AI commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

Fixed invalid GitHub Actions expression syntax in the patch installer continuous deployment workflow that was causing YAML parsing errors.

Problem

The workflow file contained malformed GitHub Actions expressions on lines 242 and 251:

if: {{ (inputs.make_release || 'true') == 'true' }}

This syntax is invalid because GitHub Actions expressions require a $ prefix before the double braces.

Solution

Updated both conditional statements to use proper GitHub Actions expression syntax:

if: ${{ (inputs.make_release || 'true') == 'true' }}

Impact

  • Before: Workflow would fail with YAML parsing errors when triggered
  • After: Workflow can parse successfully and execute conditional logic as intended
  • No functional changes: The conditional logic remains exactly the same - only the syntax has been corrected

The fix ensures the patch installer workflow can properly evaluate the make_release input parameter to conditionally run AWS credential configuration and S3 upload steps.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


This change is Reviewable

Co-authored-by: jasonleenaylor <2295227+jasonleenaylor@users.noreply.github.com>
Copilot AI changed the title [WIP] fix error in .github/workflows/patch-installer-cd.yml Fix YAML syntax errors in patch-installer-cd.yml workflow Sep 16, 2025

@jasonleenaylor jasonleenaylor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@jasonleenaylor reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @copilot)

@jasonleenaylor jasonleenaylor marked this pull request as ready for review September 16, 2025 21:27
@jasonleenaylor jasonleenaylor merged commit 6806b33 into release/9.3 Sep 16, 2025
3 checks passed
@jasonleenaylor jasonleenaylor deleted the copilot/fix-b3f109f8-31e3-41b2-ac2d-c9b213a3522f branch September 16, 2025 21:28
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