Skip to content

Test PowerShell Preview in CI - #2070

Merged
Andy Jordan (andyleejordan) merged 1 commit into
mainfrom
daily
Mar 13, 2025
Merged

Test PowerShell Preview in CI#2070
Andy Jordan (andyleejordan) merged 1 commit into
mainfrom
daily

Conversation

@andyleejordan

Copy link
Copy Markdown
Member

Replaces #1439.

@andyleejordan

Andy Jordan (andyleejordan) commented Mar 1, 2025

Copy link
Copy Markdown
Member Author

Funny day year to test this, the daily isn't available.

@andyleejordan

Copy link
Copy Markdown
Member Author

So apparently the Daily build hasn't existed in a year. I opened PowerShell/PowerShell#25124 to correct the install script, and I don't expect this to pass until it's merged as I updated it to install the preview to ~/.powershell-preview (like the Daily was) which makes way more sense.

@bergmeister

Copy link
Copy Markdown
Collaborator

Lol. But given that there are now multiple supported PowerShell versions available (LTS and current), the more valuable thing is maybe to test with those versions and the latest preview. Also, daily probably adds a bit of noise.

@andyleejordan Andy Jordan (andyleejordan) changed the title Test PowerShell daily in CI Test PowerShell Preview in CI Mar 13, 2025
@andyleejordan

Copy link
Copy Markdown
Member Author

Ah I didn't update the title. This will test the PowerShell Preview, I'm just waiting on my upstream PR to be merged so I don't have to install it to a -Destination manually.

Since the Daily no longer exists.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@bergmeister

Copy link
Copy Markdown
Collaborator

Ah I didn't update the title. This will test the PowerShell Preview, I'm just waiting on my upstream PR to be merged so I don't have to install it to a -Destination manually.

Gotcha. Makes sense. But I think it would also be useful to additionally also have one run with PowerShell 7.4 since it is LTS but the CI runners always use the latest version, which is 7.5 atm. For now we could just run test suite with PS 7.4 locally, just requires manual uninstall of newer versions because some tests call into pwsh from PATH

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants