-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Remove -Daily from install-powershell.ps1 since it no longer exists
#25124
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
base: master
Are you sure you want to change the base?
Conversation
|
Required for PowerShell/PowerShellEditorServices#2225. |
d214f23 to
31bb622
Compare
|
The bullet style was changed in order to satisfy the Markdown linter 🙃 |
|
Also required for PowerShell/PSScriptAnalyzer#2070. |
We have a request to have daily build #24566 In my understanding, daily builds are the only way for partner teams to continuously test their products. |
|
@andyleejordan should add a parameter alias to "Daily" to not break existing scripts. I'd say this merge as-is after that, but then a follow-on should replace daily with "CIBuild" or "GHBuild" to fetch the latest commit azdo build. @iSazonov per my comments on that, a "latest main commit build" is a better approach than a daily since tests are enforced for main commits. |
It depends on the needs of that project. If they want, they can track each pwsh PR and upload artifacts from it. |
Then they can still do that, and they'll get whatever the latest PR branch is available when they schedule their job to run on a daily basis. Solves both problems :) Dailies are an outdated development concept from the days where most work happened "during the day" and commit checks were rare. Since PowerShell has extensive gatekeeping on the main branch, pretty much every main commit can probably be considered to be fairly stable and unbroken at least for the vast majority of test cases, so it makes sense to provide an easy way to consume these artifacts that are already being built. |
Yeah, I'm wondering if the nicer thing to do is a) warn and just install the preview instead, or b) warn and exit with error. Right now it fails and that hasn't been breaking anyone so B is an option, but A is an option, though it's kind of a lie. |
9466126 to
66c870e
Compare
@TravisEz13 what would you prefer here and what do we want to communicate about the future of the daily build here? |
|
@TravisEz13 waiting on a decision from you / maintainers |
|
Also noting that we'll want to cleanup: |
Also install the Preview to `~/.powershell-preview` like the daily was.
66c870e to
edcd9d6
Compare
|
I never got an answer so I'm going with option A. |
|
@adityapatwardhan can you review this please? |
|
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
This removes the
-Dailyflag from theinstall-powershell.ps1script as daily builds have not been posted in a year, and @adityapatwardhan says there are no current plans to resume doing so.Also updates the script to install the Preview to
~/.powershell-previewlike the daily was.PR Context
I use this for testing PowerShell Editor Services and PSScriptAnalyzer in CI against upcoming builds of PowerShell to find bugs before they make it to release.
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.- [ ] Issue filed:
(which runs in a different PS Host).