Remove the Packaged with Windows Packaging Project (.wapproj) templates from the VSIX#6606
Open
lauren-ciha wants to merge 9 commits into
Open
Remove the Packaged with Windows Packaging Project (.wapproj) templates from the VSIX#6606lauren-ciha wants to merge 9 commits into
lauren-ciha wants to merge 9 commits into
Conversation
Member
Author
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the legacy two-project WinUI “PackagedApp” templates (the .csproj + Windows Packaging Project .wapproj approach) from the VSIX-delivered Visual Studio templates, aligning the VSIX with the single-project packaging templates that are the current recommendation.
Changes:
- Removed the C# and C++/WinRT
.wapproj-based PackagedApp template projects from the VSIX solution and manifests. - Deleted the corresponding template project files (
.csproj,.vstemplate,.wapproj, appxmanifest, and supporting content). - Updated
templates.propsto keep the VSIX-only template slot as a reserved framework hook (with no current instances).
Reviewed changes
Copilot reviewed 43 out of 70 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| dev/Templates/VSIX/WindowsAppSDK.Extension.sln | Removes PackagedApp template projects from the VSIX solution. |
| dev/Templates/VSIX/Extension/Cs/Dev17/Standalone/source.extension.vsixmanifest | Drops the C# PackagedApp template asset from the Standalone VSIX manifest. |
| dev/Templates/VSIX/Extension/Cs/Dev17/LocalDev/source.extension.vsixmanifest | Drops the C# PackagedApp template asset from the LocalDev VSIX manifest. |
| dev/Templates/VSIX/Extension/Cs/Dev17/Component/source.extension.vsixmanifest | Drops the C# PackagedApp template asset from the Component VSIX manifest. |
| dev/Templates/VSIX/Extension/Cpp/Dev17/WindowsAppSDK.Cpp.Extension.Dev17.csproj | Removes the C++ PackagedApp template project reference from the C++ VSIX project. |
| dev/Templates/VSIX/Extension/Cpp/Dev17/Standalone/source.extension.vsixmanifest | Removes the C++ PackagedApp template asset from the Standalone VSIX manifest. |
| dev/Templates/VSIX/Extension/Cpp/Dev17/LocalDev/source.extension.vsixmanifest | Removes the C++ PackagedApp template asset from the LocalDev VSIX manifest. |
| dev/Templates/VSIX/Extension/Cpp/Dev17/Component/source.extension.vsixmanifest | Removes the C++ PackagedApp template asset from the Component VSIX manifest. |
| dev/Templates/templates.props | Removes the last instance of VSIX-only templates and leaves a reserved slot comment for future use. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/WinUI.Desktop.Cs.PackagedApp.vstemplate | Deletes legacy C# PackagedApp project-group template definition. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/WinUI.Desktop.Cs.PackagedApp.csproj | Deletes legacy C# PackagedApp VS template project container. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/WapProj/WinUI.Desktop.Cs.WapProj.vstemplate | Deletes legacy C# WapProj template definition. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/WapProj/WapProjTemplate.wapproj | Deletes legacy C# Windows Packaging Project template file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/WapProj/Package-managed.appxmanifest | Deletes legacy C# packaging manifest for WapProj template. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/WinUI.Desktop.Cs.BlankApp.vstemplate | Deletes legacy C# inner “BlankApp” template used by PackagedApp. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/Properties/PublishProfiles/win-x86.pubxml | Deletes legacy publish profile used by the removed template. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/Properties/PublishProfiles/win-x64.pubxml | Deletes legacy publish profile used by the removed template. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/Properties/PublishProfiles/win-arm64.pubxml | Deletes legacy publish profile used by the removed template. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/ProjectTemplate.csproj | Deletes legacy C# inner project template used by PackagedApp. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/MainWindow.xaml.cs | Deletes legacy C# template app source file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/MainWindow.xaml | Deletes legacy C# template XAML file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/App.xaml.cs | Deletes legacy C# template app source file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/App.xaml | Deletes legacy C# template app XAML file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CSharp/PackagedApp/BlankApp/app.manifest | Deletes legacy C# template app manifest. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/WinUI.Desktop.CppWinRT.PackagedApp.vstemplate | Deletes legacy C++/WinRT PackagedApp project-group template definition. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/WinUI.Desktop.CppWinRT.PackagedApp.csproj | Deletes legacy C++/WinRT PackagedApp VS template project container. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/WapProj/WinUI.Desktop.CppWinRT.WapProj.vstemplate | Deletes legacy C++/WinRT WapProj template definition. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/WapProj/WapProjTemplate.wapproj | Deletes legacy C++/WinRT Windows Packaging Project template file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/WapProj/Package-managed.appxmanifest | Deletes legacy C++/WinRT packaging manifest for WapProj template. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/WinUI.Desktop.CppWinRT.BlankApp.vstemplate | Deletes legacy C++/WinRT inner “BlankApp” template used by PackagedApp. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/readme.txt | Deletes legacy template readme content. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/ProjectTemplate.vcxproj.filters | Deletes legacy template project filters file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/ProjectTemplate.vcxproj | Deletes legacy C++/WinRT template project file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/pch.h | Deletes legacy template precompiled header. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/pch.cpp | Deletes legacy template precompiled header source. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/MainWindow.xaml | Deletes legacy C++/WinRT template XAML file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/MainWindow.idl | Deletes legacy C++/WinRT template IDL file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/MainWindow.h | Deletes legacy C++/WinRT template header file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/MainWindow.cpp | Deletes legacy C++/WinRT template source file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/App.xaml | Deletes legacy C++/WinRT template app XAML file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/app.manifest | Deletes legacy C++/WinRT template app manifest. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/App.h | Deletes legacy C++/WinRT template app header file. |
| dev/Templates/Source/ProjectTemplates/Desktop/CppWinRT/PackagedApp/BlankApp/App.cpp | Deletes legacy C++/WinRT template app source file. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Member
Author
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Packaging a project with Windows Packaging Project (
.wapproj) is no longer the recommended way to package an app today. This PR removes the Windows Packaging Project starter template.Validation:
.\dev\Templates\VSIX\build-local-VSIX-package\build-install-localdev-vsix.ps1.\dev\Templates\test-dotnetnewTemplates.ps1 -WindowsAppSDKVersion "2.2.0" -dotnetsdkversion "8.*"A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.