Skip to content

Revert "[release/v7.6] Fetch latest ICU release version dynamically"#27019

Merged
daxian-dbw merged 1 commit intorelease/v7.6from
revert-26970-backport/release/v7.6/26827-cf40400cb
Mar 12, 2026
Merged

Revert "[release/v7.6] Fetch latest ICU release version dynamically"#27019
daxian-dbw merged 1 commit intorelease/v7.6from
revert-26970-backport/release/v7.6/26827-cf40400cb

Conversation

@daxian-dbw
Copy link
Copy Markdown
Member

Reverts #26970
This change makes it required to access an outside endpoint during packaging build, which will be blocked in our packaging pipeline due to the network isolation requirement for Compliance.

@daxian-dbw daxian-dbw requested review from a team and jshigetomi as code owners March 12, 2026 22:06
Copilot AI review requested due to automatic review settings March 12, 2026 22:06
@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 12, 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

Reverts the release-branch change that dynamically fetched the latest ICU release version during packaging, restoring an offline-friendly (network-isolated) packaging build behavior needed for compliance-restricted pipelines.

Changes:

  • Replace the dynamic ICU “latest release” lookup with a fixed $BuildICUVersion constant.
  • Remove the Get-IcuLatestRelease helper function that performed an external Invoke-WebRequest to GitHub.

@daxian-dbw daxian-dbw merged commit 322f386 into release/v7.6 Mar 12, 2026
41 of 42 checks passed
@daxian-dbw daxian-dbw deleted the revert-26970-backport/release/v7.6/26827-cf40400cb branch March 12, 2026 22:22
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.

2 participants