[PR #13535] [MERGED] Make ToolsVersion more consistent in our project files #29619

Open
opened 2026-01-31 09:35:56 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/13535
Author: @lhecker
Created: 7/18/2022
Status: Merged
Merged: 7/26/2022
Merged by: @undefined

Base: mainHead: dev/lhecker/tools-version


📝 Commits (1)

  • a809583 Make ToolsVersion more consistent in our project files

📊 Changes

117 files changed (+217 additions, -220 deletions)

View changed files

📝 build/pgo/Terminal.PGO.DB.nuspec (+1 -1)
📝 build/rules/Branding.targets (+1 -1)
📝 build/rules/CollectWildcardResources.targets (+1 -1)
📝 build/rules/GenerateFeatureFlags.proj (+1 -1)
📝 build/rules/GenerateSxsManifestsFromWinmds.targets (+1 -1)
📝 common.openconsole.props (+1 -1)
📝 custom.props (+1 -1)
📝 doc/feature_flags.md (+1 -1)
📝 samples/ConPTY/EchoCon/EchoCon/EchoCon.vcxproj (+1 -1)
📝 samples/ConPTY/GUIConsole/GUIConsole.WPF/GUIConsole.WPF.csproj (+1 -1)
📝 samples/ConPTY/MiniTerm/MiniTerm/MiniTerm.csproj (+1 -1)
📝 scratch/ScratchIslandApp/Package/Package.wapproj (+1 -1)
📝 scratch/ScratchIslandApp/SampleApp/SampleAppLib.vcxproj (+1 -1)
📝 scratch/ScratchIslandApp/SampleApp/dll/SampleApp.vcxproj (+1 -1)
📝 scratch/ScratchIslandApp/WindowExe/WindowExe.vcxproj (+1 -1)
📝 src/StaticAnalysis.ruleset (+5 -7)
📝 src/audio/midi/lib/midi.vcxproj (+1 -1)
📝 src/buffer/out/lib/bufferout.vcxproj (+1 -1)
📝 src/buffer/out/ut_textbuffer/TextBuffer.Unit.Tests.vcxproj (+1 -1)
📝 src/cascadia/CascadiaPackage/CascadiaPackage.wapproj (+1 -1)

...and 80 more files

📄 Description

While working on another PR related to this I noticed that my VS
generates .vcxproj files that are a bit different to the ones we have.
This commit is a quick search & replace of all our project files to make
(primarily) their ToolsVersion more in line with what VS does itself:
No ToolsVersion for .vcxproj, ToolsVersion="15.0"
for .csproj and ToolsVersion="4.0" for .filters files.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/13535 **Author:** [@lhecker](https://github.com/lhecker) **Created:** 7/18/2022 **Status:** ✅ Merged **Merged:** 7/26/2022 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/lhecker/tools-version` --- ### 📝 Commits (1) - [`a809583`](https://github.com/microsoft/terminal/commit/a8095837519c13a9f7698edd0a286f3602537701) Make ToolsVersion more consistent in our project files ### 📊 Changes **117 files changed** (+217 additions, -220 deletions) <details> <summary>View changed files</summary> 📝 `build/pgo/Terminal.PGO.DB.nuspec` (+1 -1) 📝 `build/rules/Branding.targets` (+1 -1) 📝 `build/rules/CollectWildcardResources.targets` (+1 -1) 📝 `build/rules/GenerateFeatureFlags.proj` (+1 -1) 📝 `build/rules/GenerateSxsManifestsFromWinmds.targets` (+1 -1) 📝 `common.openconsole.props` (+1 -1) 📝 `custom.props` (+1 -1) 📝 `doc/feature_flags.md` (+1 -1) 📝 `samples/ConPTY/EchoCon/EchoCon/EchoCon.vcxproj` (+1 -1) 📝 `samples/ConPTY/GUIConsole/GUIConsole.WPF/GUIConsole.WPF.csproj` (+1 -1) 📝 `samples/ConPTY/MiniTerm/MiniTerm/MiniTerm.csproj` (+1 -1) 📝 `scratch/ScratchIslandApp/Package/Package.wapproj` (+1 -1) 📝 `scratch/ScratchIslandApp/SampleApp/SampleAppLib.vcxproj` (+1 -1) 📝 `scratch/ScratchIslandApp/SampleApp/dll/SampleApp.vcxproj` (+1 -1) 📝 `scratch/ScratchIslandApp/WindowExe/WindowExe.vcxproj` (+1 -1) 📝 `src/StaticAnalysis.ruleset` (+5 -7) 📝 `src/audio/midi/lib/midi.vcxproj` (+1 -1) 📝 `src/buffer/out/lib/bufferout.vcxproj` (+1 -1) 📝 `src/buffer/out/ut_textbuffer/TextBuffer.Unit.Tests.vcxproj` (+1 -1) 📝 `src/cascadia/CascadiaPackage/CascadiaPackage.wapproj` (+1 -1) _...and 80 more files_ </details> ### 📄 Description While working on another PR related to this I noticed that my VS generates `.vcxproj` files that are a bit different to the ones we have. This commit is a quick search & replace of all our project files to make (primarily) their `ToolsVersion` more in line with what VS does itself: No `ToolsVersion` for `.vcxproj`, `ToolsVersion="15.0"` for `.csproj` and `ToolsVersion="4.0"` for `.filters` files. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:35:56 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#29619