-
Notifications
You must be signed in to change notification settings - Fork 521
Removes Broken "Run Pester Tests" from the Command Pallette #2729
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
Conversation
This is not quite what we want. Since this is in the "explorer/context": [ section, that controls the what shows in the context menu when you right click on file in the explorer pane (to see what I've talking about, right click on a foo.Tests.ps1 file)
What we want is two new entries under "commandPalette": [ which is right above "explorer/context": [
Also don't worry about the test failures... that's because of the Omnisharp issue.
I accidentally fixed this allowing the command palette to work so we don't need this 😅
PR Summary
Fixes #2715
Removes "PowerShell.RunPesterTestsFromFile" and "PowerShell.DebugPesterTestsFromFile" from the command pallette since they do not work, and are tasks that can effectively be run in other ways.
PR Checklist
Note: Tick the boxes below that apply to this pull request by putting an
xbetween the square brackets.Please mark anything not applicable to this PR
NA.WIP:to the beginning of the title and remove the prefix when the PR is ready