Skip to content

Fix PowerShell here-string parser error in vendor.yml#3087

Merged
DRSDavidSoft merged 1 commit into
masterfrom
copilot/fix-cicd-workflow
Jun 14, 2026
Merged

Fix PowerShell here-string parser error in vendor.yml#3087
DRSDavidSoft merged 1 commit into
masterfrom
copilot/fix-cicd-workflow

Conversation

Copilot AI commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

PowerShell requires the "@ here-string terminator to be at column 0. Inside the if/else block in Summary - Update check results, the terminators were indented 10 spaces in YAML — but the run: | block only strips 8 spaces of base indentation, leaving 2 leading spaces in the generated script and triggering:

ParserError: White space is not allowed before the string terminator.

Changes

  • vendor.yml — Replace two indented here-strings (inside if/else) with equivalent double-quoted strings using `n for newlines:
-          $summary = @"
-          ### ✅ No Updates Available
-
-          All vendor dependencies are up to date! 🎉
-          "@
+          $summary = "### ✅ No Updates Available`n`nAll vendor dependencies are up to date! 🎉`n"

Replace indented here-strings inside if/else blocks with simple
string literals. PowerShell requires the "@" terminator to be at
column 0 in the script; when inside an indented block the YAML
base-indent stripping left 2 leading spaces, causing the parser
error "White space is not allowed before the string terminator".
@DRSDavidSoft DRSDavidSoft marked this pull request as ready for review June 14, 2026 11:40
Copilot AI review requested due to automatic review settings June 14, 2026 11:40
@DRSDavidSoft DRSDavidSoft merged commit c8c8830 into master Jun 14, 2026
@DRSDavidSoft DRSDavidSoft deleted the copilot/fix-cicd-workflow branch June 14, 2026 11:40

Copilot AI 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.

Pull request overview

This PR fixes a PowerShell parsing failure in the Summary - Update check results step by eliminating indented here-strings whose terminators were not at column 0 after YAML indentation was applied.

Changes:

  • Replaced two PowerShell here-strings inside an if/else block with equivalent double-quoted strings using `n for newlines.
  • Preserved the existing summary formatting behavior while preventing ParserError: White space is not allowed before the string terminator.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants