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

@microsoft-github-policy-service microsoft-github-policy-service Bot added the No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity. label Apr 30, 2026
@microsoft-github-policy-service
Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 7 days. It will be closed if no further activity occurs within 7 days of this comment.

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 No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants