Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Document MSBuildTaskHost compatibility constraints - #13640

#13640
Merged
baronfel merged 2 commits into
maindotnet/msbuild:mainfrom
copilot/add-readme-for-visual-studio-assetsdotnet/msbuild:copilot/add-readme-for-visual-studio-assetsCopy head branch name to clipboard
Apr 28, 2026
Merged

Document MSBuildTaskHost compatibility constraints#13640
baronfel merged 2 commits into
maindotnet/msbuild:mainfrom
copilot/add-readme-for-visual-studio-assetsdotnet/msbuild:copilot/add-readme-for-visual-studio-assetsCopy head branch name to clipboard

Conversation

Copilot AI commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Context

MSBuildTaskHost is kept for Visual Studio asset compatibility for some legacy project types. It targets .NET Framework 3.5, must interoperate with the rest of MSBuild, and is not an active development area.

Changes Made

  • Added src/MSBuildTaskHost/README.md
    • Documents the compatibility purpose of the folder.
    • Calls out the .NET Framework 3.5 target.
    • Discourages style-only, modernization, cleanup, and other minor edits.

Testing

  • Not run; documentation-only change.

Notes

  • No code or build behavior changed.

Agent-Logs-Url: https://github.com/dotnet/msbuild/sessions/5f961099-f265-49ab-8d5a-0267fc3a5be1

Co-authored-by: rainersigwald <3347530+rainersigwald@users.noreply.github.com>
Comment thread src/MSBuildTaskHost/README.md Outdated
@rainersigwald
rainersigwald marked this pull request as ready for review April 28, 2026 09:12
Copilot AI review requested due to automatic review settings April 28, 2026 09:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation to clarify why src/MSBuildTaskHost exists and what constraints apply to it (legacy Visual Studio asset compatibility, .NET Framework 3.5 targeting, and interoperability requirements), helping avoid unintended “cleanup” changes in a sensitive compatibility area.

Changes:

  • Added a README describing MSBuildTaskHost’s compatibility purpose and constraints.
  • Explicitly calls out the .NET Framework 3.5 target and named-pipe interoperability requirement.
  • Advises against style-only/modernization edits unless required for compatibility/servicing.

@rainersigwald
rainersigwald requested a review from OvesN April 28, 2026 09:14
@rainersigwald rainersigwald added the Area: Our Own Build Problems affecting the build or build infrastructure of the MSBuild repo itself. label Apr 28, 2026
@baronfel
baronfel merged commit d005ac6 into main Apr 28, 2026
5 of 13 checks passed
@baronfel
baronfel deleted the copilot/add-readme-for-visual-studio-assets branch April 28, 2026 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Our Own Build Problems affecting the build or build infrastructure of the MSBuild repo itself.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.