-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add single target build ProjectCapability #4791
Conversation
@@ -104,6 +104,7 @@ | |||
<ProjectCapability Include="XamarinStaticLaunchProfiles" Condition=" '$(VisualStudioVersion)' != '' and '$(VisualStudioVersion)' < '17.0' " /> | |||
<!-- Otherwise define LaunchProfilesGroupByPlatformFilters by default --> | |||
<ProjectCapability Include="LaunchProfilesGroupByPlatformFilters" Condition=" '$(VisualStudioVersion)' == '' or '$(VisualStudioVersion)' >= '17.0' " /> | |||
<ProjectCapability Include="SingleTargetBuildForStartupProjects" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two questions:
- Did we test this works in the IDE? I worry that the capability won't be there, the way workloads work. It seems like the "outer" build doesn't load any workload MSBuild targets.
- Should we also put this in each platform workload? Like you could do a
<TargetFrameworks>net6.0-android;net6.0-ios;net6.0-windows</TargetFrameworks>
and not be using MAUI? Like a multi-targeted SkiaSharp app or something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To test this try:
<ItemGroup>
<ProjectCapability Include="DiagnoseCapabilities" />
And it should list them all in the solution tree.
@tmeschter is this something we should be doing still? |
Yes, we still need this capability for the feature to light up. |
@mattleibow can we add a condition to this to disable the include in case someone needs to back out of it? |
Description of Change
Add the capability to make use of new IDE features.