Skip to content

Fix: Wingetcreate installer count mismatch in release pipeline#568

Merged
zateutsch merged 2 commits into
mainfrom
zt/fix-winget-installer-mismatch
Jun 5, 2026
Merged

Fix: Wingetcreate installer count mismatch in release pipeline#568
zateutsch merged 2 commits into
mainfrom
zt/fix-winget-installer-mismatch

Conversation

@zateutsch
Copy link
Copy Markdown
Contributor

Winget release failed because two portable installers were added to our manifest for 0.3.1, which brought the total installer count up to 4.

Our script was only passing two url's, causing winget to error out on an installer count mismatch.

Copilot AI review requested due to automatic review settings June 3, 2026 18:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 WinGet release step in the Azure DevOps release pipeline by passing all installer URLs expected by the WinGet manifest, preventing wingetcreate from failing due to an installer count mismatch.

Changes:

  • Add x64/ARM64 portable .zip installer URLs alongside the existing x64/ARM64 .msix URLs.
  • Update logging to print all four installer URLs for easier troubleshooting.
  • Update the wingetcreate update ... --urls invocation to include all four URLs in manifest order.

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

Build Metrics Report

Binary Sizes

Artifact Baseline Current Delta
CLI (ARM64) 31.45 MB 31.45 MB ✅ 0.0 KB (0.00%)
CLI (x64) 31.79 MB 31.79 MB ✅ 0.0 KB (0.00%)
MSIX (ARM64) 13.22 MB 13.22 MB 📈 +0.1 KB (+0.00%)
MSIX (x64) 14.05 MB 14.05 MB 📉 -0.1 KB (-0.00%)
NPM Package 27.50 MB 27.50 MB 📉 -0.2 KB (-0.00%)
NuGet Package 27.59 MB 27.59 MB 📈 +0.2 KB (+0.00%)
VS Code Extension 20.32 MB 20.32 MB 📈 +0.0 KB (+0.00%)

Test Results

1192 passed, 1 skipped out of 1193 tests in 468.5s (+9.0s vs. baseline)

Test Coverage

17.2% line coverage, 36.3% branch coverage · ✅ no change vs. baseline

CLI Startup Time

40ms median (x64, winapp --version) · ✅ -5ms vs. baseline


Updated 2026-06-03 21:30:43 UTC · commit 3468e8c · workflow run

Copy link
Copy Markdown
Contributor

@Jaylyn-Barbee Jaylyn-Barbee left a comment

Choose a reason for hiding this comment

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

LGTM

@zateutsch zateutsch merged commit d6cec60 into main Jun 5, 2026
22 checks passed
@zateutsch zateutsch deleted the zt/fix-winget-installer-mismatch branch June 5, 2026 18:14
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