-
Notifications
You must be signed in to change notification settings - Fork 498
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
Update Description to reflect that PowerShell is for more than scripting #3340
Update Description to reflect that PowerShell is for more than scripting #3340
Conversation
/cc @joeyaiello @SydneyhSmith for language suggestions |
LGTM, thanks for the PR @potatoqualitee |
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.
Thanks @potatoqualitee!
@@ -271,7 +271,7 @@ function Update-Version { | |||
# TODO: Maybe cleanup the replacement logic. | |||
switch ($RepositoryName) { | |||
"vscode-powershell" { | |||
$d = "Develop PowerShell scripts in Visual Studio Code!" | |||
$d = "Develop PowerShell modules, commands and scripts in Visual Studio Code!" |
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.
Thanks for catching this 😁
@@ -141,12 +141,12 @@ task UpdatePackageJson { | |||
if ($script:IsPreviewExtension) { | |||
$script:PackageJson.name = "powershell-preview" | |||
$script:PackageJson.displayName = "PowerShell Preview" | |||
$script:PackageJson.description = "(Preview) Develop PowerShell scripts in Visual Studio Code!" | |||
$script:PackageJson.description = "(Preview) Develop PowerShell modules, commands and scripts in Visual Studio Code!" |
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.
Omg what is this script doing. @rjmholt?!
PR Summary
Updates the description to reflect that PowerShell is a powerful language. Fixes #3339.
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