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

Add PowerShell 7.8 to #requires version tests by jshigetomi · Pull Request #28402 · PowerShell/PowerShell · GitHub

Repository navigation

Add PowerShell 7.8 to #requires version tests - #28402

Open
Justin Chung (jshigetomi) wants to merge 2 commits into
masterfrom
jshigetomi-fix-packaging-tests
Open

Justin Chung (jshigetomi) wants to merge 2 commits into
masterfrom
jshigetomi-fix-packaging-tests

Conversation

Justin Chung (jshigetomi) commented Nov 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • add PowerShell 7.8 to the explicit supported-version test cases
  • make 7.9 the next unsupported minor-version test case

This fixes the cross-platform Check for version 7.8 failures seen in #28396 and other PRs after master advanced to PowerShell 7.8.

Validation

  • Confirmed the PR diff is a one-line addition of 7.8 to Requires.Tests.ps1
  • The complete test cannot pass under the locally installed PowerShell 7.6 because 7.8 is correctly unsupported there; CI runs it against the repository's PowerShell 7.8 build

Generate valid PowerShell 7 minor versions from the runtime version so the next minor is tested as unsupported without becoming stale at each release transition.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Justin Chung (jshigetomi) requested a review from a team as a code owner November 11, 2026 19:34
Copilot AI balanced review requested due to automatic review settings November 11, 2026 19:34

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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

🟢 Approval recommended

No active review issues remain for the runtime-derived version test changes.

No open issues 🎉

What changed in this PR

Updates the #requires -Version tests to follow the running PowerShell version, preventing false failures when development advances to a new release, as seen in #28396.

Changes:

  • Generates supported minor versions for the runtime’s major version instead of maintaining a fixed list.
  • Derives unsupported minor and major versions from the runtime version.
File Description
test/​powershell/​Language/​Scripting/​Requires.Tests.ps1 Replaces hardcoded version boundaries with runtime-derived test cases.

Review effort: Balanced

Add PowerShell 7.8 to the explicit supported-version list.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Justin Chung (jshigetomi) changed the title Fix #requires version tests for current release Add PowerShell 7.8 to #requires version tests Nov 11, 2026
Justin Chung (jshigetomi) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Nov 11, 2026
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

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL