-
Notifications
You must be signed in to change notification settings - Fork 44
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
Can't run in VS Code Locally #190
Comments
Looks like I'm not getting PowerShell Core installed so it's still using the inbox version. |
I used |
(Also, would be nice if we pop open the website in a tab in VS Code or something like what happens in Codespaces?) |
Before I installed PS Core separately, I was seeing the I think VS Code is programmed to detect which PowerShell versions you have and use Core if it's available, but I don't think it's coming/installing bundled together. |
That sounds plausible. The next question is "how do we fix this", because PowerShell Desktop uses |
I mean once I installed the right thing, it worked, so it may just be an extra pre-build check we need to do to let people know to install the right thing vs. just failing. (Though, I still think it's important we test and maintain our scripts to ensure they run on the Windows version of |
It's a bit of a chicken and egg situation. We need to know if the user is on Windows or Linux before picking between pwsh and powershell.exe, but we can't know that without some kind of scripting. If we can solve this problem, we're good. I wonder if Visual Studio Code has something built in that could help us? |
This should help: https://stackoverflow.com/a/64911186/5953220 |
Looks like the revamp in #91 to make everything work in codespaces broke things from working in VS Code locally due to the use of
pwsh
in thetasks.json
pre-launch step:We need to figure out a way to make this work for both local and Codespace development on Windows/Linux, there must be something to handle this built into VS Code itself, right?
On Win 10 latest.
The text was updated successfully, but these errors were encountered: