FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Test PowerShell Preview in CI by andyleejordan · Pull Request #2070 · PowerShell/PSScriptAnalyzer · GitHub

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

Copy link
Copy Markdown
Member

Replaces #1439.

Andy Jordan (andyleejordan) commented Mar 1, 2025
edited
Loading

Copy link
Copy Markdown
Member Author

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

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.

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.

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

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM!

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL