-
Notifications
You must be signed in to change notification settings - Fork 500
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
Add explorer context menus for 'Run/Debug Pester tests' #2445
Add explorer context menus for 'Run/Debug Pester tests' #2445
Conversation
@PoshChan restart ci please |
@bergmeister, all requests start with the magic word: Commands available in this repo for you:
|
@PoshChan please restart ci! |
@bergmeister, you are not authorized to request a rebuild |
you can ignore the macos failure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks for this!
@TylerLeonhardt Just something that is worthwhile noting is that it depends on the currently displayed and focused file in the editor, i.e. one needs to click on the item in explorer first and then do the right click. If a different file has the focus then that file would be chosen for the test run. Also, if someone never opened any ps1 file, then it would just silently fail because the PSIC is not up and running. |
Hmm that’s a bit unfortunate, I know I never left-click, right-click, I just right-click. |
Currently we get the path from |
I could fix it to pass the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR Summary
Similar to the existing context menu for the files tab. Manually tested that it works
cc @JustinGrote @nohwnd
PR Checklist
Note: Tick the boxes below that apply to this pull request by putting an
x
between 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