[PR #3031] Layer the globals globals on top of the root globals #25213

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

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

State: closed
Merged: Yes


Summary of the Pull Request

Layer the globals globals on top of the root globals.

PR Checklist

Detailed Description of the Pull Request / Additional comments

We added the ability for the root to be used as the globals object in #2515. However, if you have a globals object, then the settings in the root will get ignored. That's bad. We should layer them.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/3031 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request Layer the `globals` globals on top of the root globals. ## PR Checklist * [x] Closes #2906 * [x] I work here * [x] Tests added/passed * [n/a] Requires documentation to be updated ## Detailed Description of the Pull Request / Additional comments We added the ability for the root to be used as the globals object in #2515. However, if you have a globals object, then the settings in the root will get ignored. That's bad. We should layer them.
claunia added the pull-request label 2026-01-31 09:08:01 +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#25213