Skip to content
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

Fix design time build error in Roslyn.VisualStudio.Setup #77463

Conversation

jasonmalinowski
Copy link
Member

The MSBuild task invocation here is a copy/paste from earlier in the file, but the item group renamed so it didn't conflict. The rename was in complete, so the use in the Properties property was still using the old name; this had the effect that we wouldn't set the TFM property, which would cause problems for multi-targeting projects.

The MSBuild task invocation here is a copy/paste from earlier in the
file, but the item group renamed so it didn't conflict. The rename
was in complete, so the use in the Properties property was still
using the old name; this had the effect that we wouldn't set the TFM
property, which would cause problems for multi-targeting projects.
@jasonmalinowski jasonmalinowski requested a review from a team as a code owner March 7, 2025 21:26
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead labels Mar 7, 2025
Copy link
Contributor

@ToddGrun ToddGrun left a comment

Choose a reason for hiding this comment

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

:shipit:

@jasonmalinowski jasonmalinowski merged commit 2967bbe into dotnet:main Mar 7, 2025
28 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Mar 7, 2025
@jasonmalinowski jasonmalinowski deleted the fix-design-time-build-error-with-publishing branch March 7, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants