| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
PoshChan-Bot (@PoshChan) please retry windows |
Sorry, something went wrong.
|
Dongbo Wang (@daxian-dbw), successfully started retry of PowerShell-CI-Windows |
Sorry, something went wrong.
|
PoshChan-Bot (@PoshChan) please retry windows |
Sorry, something went wrong.
|
Dongbo Wang (@daxian-dbw), successfully started retry of PowerShell-CI-Windows |
Sorry, something went wrong.
|
PoshChan-Bot (@PoshChan) please retry windows |
Sorry, something went wrong.
|
Dongbo Wang (@daxian-dbw), successfully started retry of PowerShell-CI-Windows |
Sorry, something went wrong.
|
Aditya Patwardhan (@adityapatwardhan) I think this PR is ready to be merged. |
Sorry, something went wrong.
|
🎉v7.0.0-rc.1 has been released which incorporates this pull request.:tada: Handy links: |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
PR Summary
Update the feature to support the notification type that can be configured using the environment variable POWERSHELL_UPDATECHECK.
This is to address the feedback from powershell committee in the RFC review: PowerShell/PowerShell-RFC#162 (comment), quoted below:
POWERSHELL_UPDATECHECK accepts 3 values: Off, Default, and LTS (case-insensitive), and the behavior is as follows:
For a specific version of pwsh and a specific notification type, only one update check task, at most, will run to complete per a day. Other tasks should be able to detect "a check is in progress" or "the check has been done for today" and bail out early, to avoid any unnecessary network IO or CPU cycles.
Note that this is a breaking change comparing to preview-5 and preview-6, because the environment variable name and its accepted values are changed.
PR Context
PR Checklist