[PR #19605] Tidy up some of our Velocity features #31890

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

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

State: closed
Merged: Yes


A few features were marked "always disabled" and then enabled in dev, canary and preview. I simplified those to "always enabled" and disabled in release instead. This required changes to Generate-FeatureStaging to make it consider WindowsInbox a Release branding (which, honestly, it always should have been.)

  • Feature_DynamicSSHProfiles
  • Feature_ShellCompletions
  • Feature_SaveSnippet
  • Feature_QuickFix

Feature_DynamicSSHProfiles had been improperly gating access to the SSH profile generator, which caused the build to fail when the feature was actually disabled after #19239.

Feature_DisableWebSourceIcons was deprecated in #19143, but the XML file never got the memo.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/19605 **State:** closed **Merged:** Yes --- A few features were marked "always disabled" and then enabled in dev, canary and preview. I simplified those to "always enabled" and disabled in release instead. This required changes to Generate-FeatureStaging to make it consider `WindowsInbox` a Release branding (which, honestly, it always should have been.) - Feature_DynamicSSHProfiles - Feature_ShellCompletions - Feature_SaveSnippet - Feature_QuickFix Feature_DynamicSSHProfiles had been improperly gating access to the SSH profile generator, which caused the build to fail when the feature was actually disabled after #19239. Feature_DisableWebSourceIcons was deprecated in #19143, but the XML file never got the memo.
claunia added the pull-request label 2026-01-31 09:50:12 +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#31890