-
Notifications
You must be signed in to change notification settings - Fork 8.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Terminal defaults to powershell ignoring settings #5578
Comments
Ok, I just had a look at the release notes and realized it's supposed to work this way... https://github.com/microsoft/terminal/releases
But let me say, since pretty much all of us have this on autoupdate from the Windows store, this very far from the best user experience, especially for people who have been part of the alpha since the beginning (yours truly). Suddenly the app updates, you might not even realize it and... boom, part of your settings aren't used anymore and you head to github to file for a nasty bug... I'd suggest doing SOMETHING to notify users of this BREAKING change, otherwise this makes me feel pretty stupid... For example? Open a window on the first launch showing the DAMN release notes... that might be a nice approach IMHO. It's very similar to what's used in... let me think of a modern example... uhm have you heard about the little app called VsCode ? Sorry for the sarcasm, I love your work and would love even more if you paid more attention to this kind of things. Thank you for your time and patience. S. |
Hopefully this non-existent issue can help other ppl by pointing to the release notes: https://github.com/microsoft/terminal/releases/tag/v0.11.1121.0 The Problem
more infos here: The FixDiscard globals and move them to the json root: After that it's good. Have fun! |
good job. Solved my problem. |
Hey @simmessa
Thanks for being with us so long. I appreciate that this has been a terrible obnoxious and sudden change for everyone, and I’m really sorry about the communication—or lack thereof—here. One of the things that we’re hoping to roll out after we launch 1.0 is some sort of update notification with release notes (#5513, colorfully-titled by our community). We ended up batching all the breaking changes up until the end of the preview cycle, but I’d like to assure you that post-v1 settings changes will be accompanied with migrators, notifications or just good old-fashioned backwards compatibility. |
Hi Dustin, sure, no problem, please forgive my rant, it's nice to know you're working on #5513. That kind of functionality will surely make things much smoother for all of us! Again, you've done some outstanding job and now that V1 is here I can't wait to put my CLI hands on it :) |
@simmessa I think that, in addition to showing the message, they could have just migrated it automatically |
@CrazyShot yes that would have been ideal but...guess they also had a lot of better /more useful things to do :D |
Environment
Steps to reproduce
Just open Windows Terminal, and the starting shell is powershell, regardless of "defaultProfile" setting in global (settings.json).
I don't really know of any other things that might trigger this.
Expected behavior
Windows Terminal should just honor my settings and open Ubuntu as default profile (the guid is specified in defaultProfile).
Actual behavior
A powershell prompt is started
The text was updated successfully, but these errors were encountered: