Skip to content

Fix #6452: dotnet new winui-mvvm now creates Properties\PublishProfiles\#6457

Merged
DinahK-2SO merged 1 commit intouser/muyuanli/dotnetnewtemplatefrom
user/DinahK-2SO/fix-mvvm-pubxml
May 6, 2026
Merged

Fix #6452: dotnet new winui-mvvm now creates Properties\PublishProfiles\#6457
DinahK-2SO merged 1 commit intouser/muyuanli/dotnetnewtemplatefrom
user/DinahK-2SO/fix-mvvm-pubxml

Conversation

@DinahK-2SO
Copy link
Copy Markdown
Contributor

@DinahK-2SO DinahK-2SO commented May 6, 2026

This is a fix for

Adding the pubxml files.

Verification

dotnet pack dev/Templates/Dotnet/WinAppSdk.CSharp.DotnetNewTemplates.csproj -c Release -o localpackages
# nupkg now contains:
#   content/winui.mvvmapp/Properties/PublishProfiles/win-arm64.pubxml
#   content/winui.mvvmapp/Properties/PublishProfiles/win-x64.pubxml
#   content/winui.mvvmapp/Properties/PublishProfiles/win-x86.pubxml

dotnet new install ./localpackages/Microsoft.WindowsAppSDK.WinUI.CSharp.Templates.<version>.nupkg
dotnet new winui-mvvm -n TestMvvm
# TestMvvm/Properties/PublishProfiles/ now exists with the 3 .pubxml files.

The MvvmApp template referenced three publish profiles via `<ProjectItem>`
entries in WinUI.Desktop.Cs.MvvmApp.vstemplate, and the dotnet new package
glob expected them to ship as part of the `**\*.*` pattern, but the source
folder dev/Templates/Source/ProjectTemplates/Desktop/CSharp/MvvmApp/Properties/
PublishProfiles/ never existed on disk. As a result:

* `dotnet new winui-mvvm` produced a project missing the entire
  Properties\PublishProfiles\ folder.
* VS-instantiated projects from the same source were missing them too
  (latent bug fixed at the same time).

The pubxml files for the other four packaged templates (single-project,
navigation, tabview, unit-test) are content-identical -- pure publish
configuration with `<PublishDir>`, `<RuntimeIdentifier>`, and
`<SelfContained>true</SelfContained>`. This change copies the same three
files into MvvmApp's source folder.

Note: a global `*.pubxml` pattern in .gitignore (line 195) ignores these
files by default. The other four templates' pubxml files were force-added
the same way; staying consistent here.
@DinahK-2SO DinahK-2SO merged commit 2007494 into user/muyuanli/dotnetnewtemplate May 6, 2026
1 check passed
@DinahK-2SO DinahK-2SO deleted the user/DinahK-2SO/fix-mvvm-pubxml branch May 6, 2026 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant