Skip to content

Add a version suffix to the Windows Powershell profile#20135

Open
lhecker wants to merge 1 commit intomainfrom
dev/lhecker/pwsh-5.1
Open

Add a version suffix to the Windows Powershell profile#20135
lhecker wants to merge 1 commit intomainfrom
dev/lhecker/pwsh-5.1

Conversation

@lhecker
Copy link
Copy Markdown
Member

@lhecker lhecker commented Apr 21, 2026

As explained in the diff. Part of MSFT-59472435.

Validation Steps Performed

  • Launched WT Dev with an existing settings.json
  • Name migrated ✅

Copy link
Copy Markdown
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have some bad news: that test in CommandlineTest might cover some load bearing use cases...

@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Apr 23, 2026

// Exclude some false positives from userDefaults.json
// NOTE: we can't use the OriginTag here because it hasn't been set yet!
const bool isWinPow = _Guid.has_value() && *_Guid == DEFAULT_WINDOWS_POWERSHELL_GUID; //_Name.has_value() && til::equals_insensitive_ascii(*_Name, L"Windows PowerShell");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder why we kept this commented out code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants