-
Notifications
You must be signed in to change notification settings - Fork 511
Enable ISE Mode command fails in Azure Data Studio #3709
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
Comments
Hi, you should not be "Adding the Extension PowerShell v2021.8.2 (current)." to Azure Data Studio, it comes with a bundled and tested PowerShell extension. Can you reinstall ADS and use the bundled extension? Having multiple copies of an extension can cause issues, especially when they try to set each other's settings (like the Enable ISE Mode command does). |
@SQLvariant / @Charles-Gagnon I think this user may be hitting a bug in the old version of the extension (though I'm not entirely sure, I also suspect there's some weird state with two extensions being installed). Anyway, as ADS has four month old copy, could you please pull in v2021.10.2? |
Would be glad to! Typically in the past you've had some sort of automated pipeline that opens up the PR to update the gallery, is that still active? I can do it manually if needed but we generally prefer customers maintain their own extensions when possible 😁 |
@Charles-Gagnon In our last sync we all decided to disable that as it was broken and difficult to maintain, and ADS would just manually check if there's a new release when doing updates. I suggested automating opening an issue, but at the time you didn't want that. That suggestion is still open though! CC: @StevenBucher98 (Did you happen to take notes in that meeting?) |
Ah thanks, I recall that now. Yeah opening an issue seems like a good idea for now then, at least serve as a reminder of what to do. Did you want me to handle updating the gallery this time or is someone on your team able to take care of it? |
Maybe I'm not recalling correctly but I thought you'd offered to update your gallery, at least until such time that the ADS gallery became updatable by third-parties. I thought there was some pending work to make submissions easier. I can setup a step in our release process to open an issue against https://github.com/microsoft/azuredatastudio/issues when we do a stable release. |
I can do the updates if you don't have resources yourself to do it, but we generally prefer that extension owners handle updating their own extensions (trying to keep track of updates of all the various partners gets overwhelming quickly). |
PR to update to 2021.10.2 microsoft/azuredatastudio#17931 |
@williammsft Are you able to update via the ADS gallery to v2021.10.2, and if so, did it resolve your issue? |
@andschwa I did try going to v2021.10.2, still seeing the issue there when running the PowerShell: Enable ISE Mode. I even tried the vsix instead of using the market place one and that has not helped either. |
This turned out to be an issue related to the Azure Data Studio itself. |
Prerequisites
Summary
After installing the PowerShell Extension for Azure Data Studio and then attempting to enable the ISE mode an error comes up with "Command 'PowerShell: Enable ISE Mode' resulted in an error (Running the contributed command: "PowerShell:EnableISEMode' failed.)". This is preventing ISE mode from fully loading and being useable. All that needs to be done to reproduce this with the PowerShell Extension installed is running the PowerShell: Enable ISE Mode.
I would expect this to be similar to Visual Studio Code after launching. The version being reported by code --list-extensions --show-versions | Select-String powershell in Azure Data Studio is the same as the latest for Visual Studio Code, but the download reports 2021.8.2.
PowerShell Version
Visual Studio Code Version
Extension Version
[email protected]
Steps to Reproduce
Result:
Visuals
No response
Logs
No response
The text was updated successfully, but these errors were encountered: