-
-
Notifications
You must be signed in to change notification settings - Fork 446
plugin/shell: support Pwsh (PowerShell 7.x) #2183
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
plugin/shell: support Pwsh (PowerShell 7.x) #2183
Conversation
enables us to add new `Shell` options without messing with the user's saved settings, while keeping the `RunCommand` option last.
Co-authored-by: shobu13 <[email protected]>
@check-spelling-bot Report🔴 Please reviewSee the 📂 files view or the 📜action log for details. Unrecognized words (1)pwsh To accept ✔️ these unrecognized words as correct and remove the previously acknowledged and now absent words, run the following commands... in a clone of the [email protected]:JohnTheGr8/Flow.Launcher.git repository curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/main/apply.pl' |
perl - 'https://github.com/Flow-Launcher/Flow.Launcher/actions/runs/5245606870/attempts/1' Errors (1)See the 📂 files view or the 📜action log for details.
See ❌ Event descriptions for more information. If the flagged items are 🤯 false positivesIf items relate to a ...
|
yes please. +1 for ETA: Alternatively, perhaps:
|
Maybe let's also add an option to add custom shell? Other shell is also pretty good like nushell. One concern of this is that pwsh is not installed in windows by default. I think a textbox to customize shell might be better? |
Sure, but let's do that in a different PR. Could be added along with Windows Terminal support.
That's a good point... should we handle this or trust that the user knows not to pick
Can't just do that, each shell has different CLI options for no-exit, passing the sub-command to execute, etc |
to add to this: I think it would be best if we add Windows Terminal support and implement profile selection, as discussed in #1116. Then users can configure all their preferences from Terminal (what shell to use, etc) and Flow just uses that without complicating things on our end... |
Add
Pwsh
as a shell option. Powershell users would probably prefer to use Pwsh over the old(Windows) Powershell
that ships with Windows.This raises everyone's favorite problem: how to name each case to distinguish between v5 (
Windows Powershell
) and v7.x (Powershell (Core)
orPwsh
). Thoughts welcome...( changes partly picked from #1116 )