[PR #12674] [MERGED] Update .NET from unsupported 4.5 to 4.8 to silence VS2022 #29172

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/12674
Author: @dmachaj
Created: 3/11/2022
Status: ✅ Merged
Merged: 3/14/2022
Merged by: @undefined

Base: main ← Head: user/dmachaj/fix-dotnet-error


📝 Commits (2)

  • c0322d7 Update .net framework from unsupported 4.5 to 4.8 to silence VS2022 errors when opening solution
  • ba35f23 defuse the BOM

📊 Changes

6 files changed (+34 additions, -8 deletions)

View changed files

📝 src/cascadia/WindowsTerminal_UIATests/WindowsTerminal.UIA.Tests.csproj (+2 -1)
📝 src/cascadia/WindowsTerminal_UIATests/app.config (+4 -1)
📝 src/host/ft_uia/Host.Tests.UIA.csproj (+2 -1)
📝 src/host/ft_uia/app.config (+4 -1)
📝 src/tools/vtapp/App.config (+1 -1)
📝 src/tools/vtapp/VTApp.csproj (+21 -3)

📄 Description

Summary of the Pull Request

Closes #12670
Visual Studio 2022 offered to upgrade .NET Framework from an unsupported 4.5 to a supported 4.8. I let it do its thing (and undid a number of whitespace-only changes that aren't necessary). The project still builds after this. The UIA tests fail to run but I think that is preexisting and will be filing a new issue momentarily.

PR Checklist

Validation Steps Performed

The solution compiles in VS2022 (except for #12673).


🔄 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/12674 **Author:** [@dmachaj](https://github.com/dmachaj) **Created:** 3/11/2022 **Status:** ✅ Merged **Merged:** 3/14/2022 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `user/dmachaj/fix-dotnet-error` --- ### 📝 Commits (2) - [`c0322d7`](https://github.com/microsoft/terminal/commit/c0322d71250bd3e5b5c754f4396b47d5d5bb7c17) Update .net framework from unsupported 4.5 to 4.8 to silence VS2022 errors when opening solution - [`ba35f23`](https://github.com/microsoft/terminal/commit/ba35f23274dbd4ba85793f18afca6a143706c934) defuse the BOM ### 📊 Changes **6 files changed** (+34 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/WindowsTerminal_UIATests/WindowsTerminal.UIA.Tests.csproj` (+2 -1) 📝 `src/cascadia/WindowsTerminal_UIATests/app.config` (+4 -1) 📝 `src/host/ft_uia/Host.Tests.UIA.csproj` (+2 -1) 📝 `src/host/ft_uia/app.config` (+4 -1) 📝 `src/tools/vtapp/App.config` (+1 -1) 📝 `src/tools/vtapp/VTApp.csproj` (+21 -3) </details> ### 📄 Description ## Summary of the Pull Request Closes #12670 Visual Studio 2022 offered to upgrade .NET Framework from an unsupported 4.5 to a supported 4.8. I let it do its thing (and undid a number of whitespace-only changes that aren't necessary). The project still builds after this. The UIA tests fail to run but I think that is preexisting and will be filing a new issue momentarily. ## PR Checklist * [x] Closes #12670 * [x] Tests added/passed ## Validation Steps Performed The solution compiles in VS2022 (except for #12673). --- <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:33:15 +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#29172