Skip to content

Fix vendor workflow failure when exporting empty GitHub env vars#3086

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

Fix vendor workflow failure when exporting empty GitHub env vars#3086
DRSDavidSoft merged 1 commit into
masterfrom
copilot/fix-ci-workflow-error

Conversation

Copilot AI commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

The Update Vendor workflow failed in Checking for updates when multiple dependencies were updated. The job attempted to export intentionally empty SINGLE_DEP_* values, but Set-GHVariable rejected empty-string input and terminated the run.

  • What changed

    • Updated scripts/utils.ps1 to allow empty-string values in Set-GHVariable.
    • Kept existing call sites and workflow behavior unchanged; only the helper’s parameter binding was corrected.
  • Why this resolves the failure

    • Multi-update paths legitimately set empty placeholders for single-dependency metadata.
    • Allowing empty-string input prevents a hard failure while preserving downstream conditional logic.
  • Code change

function Set-GHVariable {
    param(
        [Parameter(Mandatory = $true)]
        [string]$Name,
        [Parameter(Mandatory = $true)]
        [AllowEmptyString()]
        [string]$Value
    )
    ...
}

@DRSDavidSoft DRSDavidSoft marked this pull request as ready for review June 14, 2026 11:25
Copilot AI review requested due to automatic review settings June 14, 2026 11:25

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 the Update Vendor GitHub Actions workflow failing when it tries to export intentionally empty SINGLE_DEP_* environment variables by adjusting the PowerShell helper Set-GHVariable to accept empty-string values.

Changes:

  • Added [AllowEmptyString()] to the mandatory Value parameter in Set-GHVariable so empty-string exports don’t fail parameter binding.

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

@DRSDavidSoft DRSDavidSoft merged commit 9ad356a into master Jun 14, 2026
1 check passed
@DRSDavidSoft DRSDavidSoft deleted the copilot/fix-ci-workflow-error branch June 14, 2026 11:30
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