[PR #3931] [MERGED] Add some issue numbers for TODOs from #3468 #25558

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/3931
Author: @zadjii-msft
Created: 12/12/2019
Status: Merged
Merged: 12/12/2019
Merged by: @undefined

Base: masterHead: dev/migrie/b/minor-retro-cleanup


📝 Commits (2)

  • e37baee This stuff wasn't important enough to block the PR over
  • 9ac7df2 whoops missed this

📊 Changes

5 files changed (+11 additions, -9 deletions)

View changed files

📝 doc/cascadia/SettingsSchema.md (+1 -1)
📝 doc/cascadia/profiles.schema.json (+4 -4)
📝 src/cascadia/TerminalControl/TermControl.cpp (+2 -0)
📝 src/renderer/dx/DxRenderer.cpp (+1 -1)
📝 src/renderer/dx/ScreenPixelShader.h (+3 -3)

📄 Description

Summary of the Pull Request

The original PR had a few TODOs in it without issue numbers. IMO, this wasn't important enough to block the PR over. Also I'm impatient and wanted that setting.

After I merged the PR I created the issues and added the numbers myself.

References

PR Checklist

  • Closes nothing, this just adds a couple TODOs
  • I work here
  • this really doesn't need tests
  • This is a docs update

🔄 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/3931 **Author:** [@zadjii-msft](https://github.com/zadjii-msft) **Created:** 12/12/2019 **Status:** ✅ Merged **Merged:** 12/12/2019 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `dev/migrie/b/minor-retro-cleanup` --- ### 📝 Commits (2) - [`e37baee`](https://github.com/microsoft/terminal/commit/e37baee1dcc650a7232e53f8e3405b24d21683b1) This stuff wasn't important enough to block the PR over - [`9ac7df2`](https://github.com/microsoft/terminal/commit/9ac7df23d612049db52218b5ad91b52c3f8eb0a8) whoops missed this ### 📊 Changes **5 files changed** (+11 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `doc/cascadia/SettingsSchema.md` (+1 -1) 📝 `doc/cascadia/profiles.schema.json` (+4 -4) 📝 `src/cascadia/TerminalControl/TermControl.cpp` (+2 -0) 📝 `src/renderer/dx/DxRenderer.cpp` (+1 -1) 📝 `src/renderer/dx/ScreenPixelShader.h` (+3 -3) </details> ### 📄 Description ## Summary of the Pull Request The original PR had a few TODOs in it without issue numbers. IMO, this wasn't important enough to block the PR over. _Also I'm impatient and wanted that setting_. After I merged the PR I created the issues and added the numbers myself. ## References ## PR Checklist * [x] Closes nothing, this just adds a couple TODOs * [x] I work here * [x] this _really_ doesn't need tests * [x] This _is_ a docs update --- <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:10:18 +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#25558