[PR #10907] Add logging, test for #10875 #28302

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

Original Pull Request: https://github.com/microsoft/terminal/pull/10907

State: closed
Merged: Yes


Summary of the Pull Request

This isn't a fix for #10875, but it is logging that help identify the root cause here. The logging may additionally be helpful for some of the other issues we're seeing elsewhere in the repo, namely #10340.

@lhecker is actually working on the fix for #10875, so hopefully this test will help validate.

References

PR Checklist

  • Closes nothing
  • I work here
  • Tests added, and they absolutely fail, but they're localtests, so ¯\_(ツ)_/¯
  • [n/a] Requires documentation to be updated

details

While I was here, I noticed that KeyBindingsTests::KeyChords has been broken for some time now. So I fixed that too.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/10907 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request This isn't a fix for #10875, but it is logging that help identify the root cause here. The logging may additionally be helpful for some of the other issues we're seeing elsewhere in the repo, namely #10340. @lhecker is actually working on the fix for #10875, so hopefully this test will help validate. ## References * Regressed in #10666. * logging for #8888 ## PR Checklist * [x] Closes nothing * [x] I work here * [x] Tests added, and they absolutely fail, but they're localtests, so ¯\\\_(ツ)_/¯ * [n/a] Requires documentation to be updated ## details While I was here, I noticed that `KeyBindingsTests::KeyChords` has been broken for some time now. So I fixed that too.
claunia added the pull-request label 2026-01-31 09:27:38 +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#28302