-
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
added functionality to install the VSCode context menus. #1238
Conversation
… is now always added to the search path
This is great! Thanks for contributing 😃 FYI, we got another PR that touched this file so their may be a very small merge conflict coming your way. Changelog stuff. When you get a chance, sign the CLA so we can get this in 👍 |
@detlefs did you want to resolve the conflict so we can get this in :) |
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.
We just need to resolve the conflicts and we can merge this.
Although this is stale, this would still be nice to have. Maybe we could resolve the conflicts ourselves and merge in? |
scripts/Install-VSCode.ps1
Outdated
@@ -202,4 +213,4 @@ if (!($IsLinux -or $IsOSX)) { | |||
} | |||
else { | |||
Write-Error "This script is currently only supported on the Windows operating system." | |||
} | |||
} |
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.
Should probably add back in the final newline.
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.
Other than the missing final newline, this LGTM.
* Add functionality to install the VSCode context menus. * VSCode is now always added to the search path
Ooo need to add this to the signing process so we can release this on the Gallery |
added functionality to install the VSCode context menus. Also, VSCode is now always added to the search path