Skip to content

#388: Exclude AppX from package and show warning#389

Merged
nmetulev merged 4 commits intomainfrom
zt/388-ignore-AppX-pack
Apr 3, 2026
Merged

#388: Exclude AppX from package and show warning#389
nmetulev merged 4 commits intomainfrom
zt/388-ignore-AppX-pack

Conversation

@zateutsch
Copy link
Copy Markdown
Contributor

@zateutsch zateutsch commented Apr 3, 2026

Fixes #388

Update MsixService to exclude AppX folder from msix packages.

Also fixed a mismatch command name and broken link in README.

@zateutsch zateutsch requested a review from nmetulev April 3, 2026 18:41
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

Build Metrics Report

Binary Sizes

Artifact Baseline Current Delta
CLI (ARM64) 28.46 MB 28.46 MB ✅ 0.0 KB (0.00%)
CLI (x64) 28.91 MB 28.91 MB 📈 +0.5 KB (+0.00%)
MSIX (ARM64) 12.05 MB 12.05 MB 📈 +0.5 KB (+0.00%)
MSIX (x64) 12.80 MB 12.81 MB 📈 +0.5 KB (+0.00%)
NPM Package 25.07 MB 25.07 MB 📈 +3.0 KB (+0.01%)
NuGet Package 25.15 MB 25.15 MB 📈 +2.0 KB (+0.01%)

Test Results

647 passed out of 647 tests in 470.1s (+47.9s vs. baseline)

Test Coverage

30.8% line coverage, 42.4% branch coverage · ✅ no change vs. baseline

CLI Startup Time

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


Updated 2026-04-03 19:18:19 UTC · commit 4737d6d · workflow run

@nmetulev nmetulev enabled auto-merge (squash) April 3, 2026 19:00
@nmetulev nmetulev merged commit f2b0d32 into main Apr 3, 2026
11 checks passed
@nmetulev nmetulev deleted the zt/388-ignore-AppX-pack branch April 3, 2026 19:18
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.

[BUG] Pack command should ignore AppX folder

2 participants