Skip to content

[release/v7.6] Fetch latest ICU release version dynamically#26970

Merged
daxian-dbw merged 1 commit intoPowerShell:release/v7.6from
daxian-dbw:backport/release/v7.6/26827-cf40400cb
Mar 9, 2026
Merged

[release/v7.6] Fetch latest ICU release version dynamically#26970
daxian-dbw merged 1 commit intoPowerShell:release/v7.6from
daxian-dbw:backport/release/v7.6/26827-cf40400cb

Conversation

@daxian-dbw
Copy link
Copy Markdown
Member

Backport of #26827 to release/v7.6

Triggered by @daxian-dbw on behalf of @kasperk81

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Updates build system to dynamically fetch the latest ICU release version from GitHub instead of using hardcoded values. This simplifies build maintenance and ensures latest ICU versions are used.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Verified by running build process to confirm ICU version is fetched correctly from GitHub releases API. No changes to runtime behavior or test requirements.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Medium risk as it changes build system behavior for ICU version detection. The change makes builds more maintainable by fetching latest ICU releases dynamically rather than hardcoding versions. Limited scope to build-time operations only.

@daxian-dbw daxian-dbw requested review from a team and jshigetomi as code owners March 9, 2026 22:24
Copilot AI review requested due to automatic review settings March 9, 2026 22:24
@daxian-dbw daxian-dbw added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Mar 9, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Backports the build/packaging change to stop hardcoding the ICU “build” major version and instead derive it dynamically from the latest ICU GitHub release, keeping dependency generation up to date over time.

Changes:

  • Replaces the hardcoded ICU build major version with a call to Get-IcuLatestRelease.
  • Adds Get-IcuLatestRelease to fetch and parse the latest ICU release info from GitHub.

@kasperk81
Copy link
Copy Markdown
Contributor

one of those times when you can safely ignore copilot's word salad

@daxian-dbw daxian-dbw merged commit f912f2b into PowerShell:release/v7.6 Mar 9, 2026
42 checks passed
@daxian-dbw daxian-dbw deleted the backport/release/v7.6/26827-cf40400cb branch March 9, 2026 23:28
daxian-dbw added a commit that referenced this pull request Mar 12, 2026
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.

4 participants