-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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 install-powershell.ps1 with more options #19884
base: master
Are you sure you want to change the base?
Update install-powershell.ps1 with more options #19884
Conversation
Minor additional formatting changes were included whilst editing the script but does not impact use Tested with the following which all work as expected .\tools\install-powershell.ps1 -Destination C:\ps -Version '7.3.1' -Verbose Unzips to C:\ps-7.3.1\ and informs the user on this .\tools\install-powershell.ps1 -Version 'v7.3.1' -Verbose Unzips to $env:LOCALAPPDATA\Microsoft\powershell-7.3.1\ and informs the user on this .\tools\install-powershell.ps1 -Destination C:\ps -LTS -Verbose Unzips to C:\ps-LTS\ and informs the user on this |
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
@kilasuit The "PowerShell-CI-install-ps" job failed, which is likely caused by the changes in this PR. Please take a look. |
@daxian-dbw I have looked and can't see why my changes could cause the jobs to fail (minor edit) @TravisEz13 or at least that I think I can fix as the error message is pointing to an issue as listed below (hope saves you from digging)
|
Currently working on this with updating the parameter sets properly - will push another update with this in the coming days |
@TravisEz13 - have added tests for this using Windows agents for all the options that I've added, which also includes an explicit stable option which is only slightly different end install location & doesn't break how it works currently when run like |
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
2 similar comments
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
@TravisEz13 Please review again when you get the time. Thanks! |
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
1 similar comment
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
PR Summary
Adds additional parameters into the install-powershell.ps1 script for use with installing LTS or a specific version as passed through
Also updates the readme for this script.
Fixes #15403
Fixes #19883
PR Context
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).