[PR #10714] [MERGED] Don't auto-generate the hidden field when creating profile stubs #28197

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/10714
Author: @PankajBhojwani
Created: 7/19/2021
Status: Merged
Merged: 7/21/2021
Merged by: @undefined

Base: mainHead: dev/pabhoj/generated_hidden


📝 Commits (1)

📊 Changes

1 file changed (+0 additions, -2 deletions)

View changed files

📝 src/cascadia/TerminalSettingsModel/Profile.cpp (+0 -2)

📄 Description

Summary of the Pull Request

We no longer automatically write the 'hidden' field for profile stubs we create

Note: This does not retroactively remove the automatically generated hidden fields in current settings files

PR Checklist

Validation Steps Performed

Deleted the ubuntu stub in my settings file, booted up terminal, new created stub did not have the hidden field. Created a fragment that overrides the hidden field and it worked.


🔄 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/10714 **Author:** [@PankajBhojwani](https://github.com/PankajBhojwani) **Created:** 7/19/2021 **Status:** ✅ Merged **Merged:** 7/21/2021 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/pabhoj/generated_hidden` --- ### 📝 Commits (1) - [`59681f5`](https://github.com/microsoft/terminal/commit/59681f5429691a43bc1521f87fc168fc002bd5dd) don't generate hidden ### 📊 Changes **1 file changed** (+0 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalSettingsModel/Profile.cpp` (+0 -2) </details> ### 📄 Description ## Summary of the Pull Request We no longer automatically write the 'hidden' field for profile stubs we create **Note**: This does not retroactively remove the automatically generated hidden fields in current settings files ## PR Checklist * [x] Closes #10539 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx * [ ] Schema updated. * [x] I work here ## Validation Steps Performed Deleted the ubuntu stub in my settings file, booted up terminal, new created stub did not have the hidden field. Created a fragment that overrides the hidden field and it worked. --- <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:26: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#28197