Terminal: Misspelled "PowerShell" in defaults.json and settings.json files

Created on 27 Apr 2020  路  3Comments  路  Source: microsoft/terminal

Could PowerShell be properly capitalized in the defaults.json and settings.json file by GA release?

"Campbell Powershell" should be "Campbell PowerShell"

"Windows.Terminal.PowershellCore" should be "Windows.Terminal.PowerShellCore"

Actually, "Core" is now gone. In 0.11 Terminal team even removed 7 and is using just "PowerShell", so source should be "Windows.Terminal.PowerShell".

Environment

Windows build number: 
Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.18363.0 Microsoft Windows NT 10.0.18363.0

Windows Terminal version:
Version: 0.11.1121.0

Steps to reproduce


Open auto-generated defaults.json and settings.json files.

Expected behavior


"name": "Campbell PowerShell"
"source": "Windows.Terminal.PowerShell"

Actual behavior


"name": "Campbell Powershell"
"source": "Windows.Terminal.PowershellCore"

Needs-Tag-Fix Resolution-Won't-Fix

Most helpful comment

Windows Terminal is not shipped. It's still in preview. You broke my profiles.json in 0.11, and you haven't heard me complaining about that. Because it's a preview release. And I expect breaking changes. Not fixing wrong capitalization bugs before GA is just sloppy. Sorry.

All 3 comments

Thanks for the report! Unfortunately, the source value is a contract between the user settings and Terminal, and it was created back when Core was part of powershell's name. It's not negotiable now that we've shipped it.

As for the color scheme name, I reckon we could fix it -- that color scheme is also deprecated, and only intended to be used by the inbuilt Windows PowerShell profile.

I don't disagree that these are _wrong_, but I'm going to mark this one as Won't fix. It's not worth the compatibility issue we'll probably cause with our future selves 馃槮

Windows Terminal is not shipped. It's still in preview. You broke my profiles.json in 0.11, and you haven't heard me complaining about that. Because it's a preview release. And I expect breaking changes. Not fixing wrong capitalization bugs before GA is just sloppy. Sorry.

If it were just you and just your opinion, I鈥檇 be sold on it, you know? But the number of duplicates on #5458 and the sheer volume of dissatisfaction _even though we鈥檙e in preview_ has made us more than a little shy about making further breaking changes.. especially mere weeks before we do final signoffs. I鈥檓 happy to fix the scheme name, but the generator may just have to stay. Sorry.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

TayYuanGeng picture TayYuanGeng  路  3Comments

mdtauk picture mdtauk  路  3Comments

mdtauk picture mdtauk  路  3Comments

NickITGuy picture NickITGuy  路  3Comments

zadjii-msft picture zadjii-msft  路  3Comments