Skip to content

[PERF] Remove HybridGlobalization from perf-ios-scenarios-build-jobs.yml#120338

Merged
LoopedBard3 merged 1 commit intodotnet:mainfrom
LoopedBard3:PerfRemoveLastiOSHGRemnants
Oct 3, 2025
Merged

[PERF] Remove HybridGlobalization from perf-ios-scenarios-build-jobs.yml#120338
LoopedBard3 merged 1 commit intodotnet:mainfrom
LoopedBard3:PerfRemoveLastiOSHGRemnants

Conversation

@LoopedBard3
Copy link
Member

Remove HybridGlobalization from perf-ios-scenarios-build-jobs.yml, the last place in the performance repo testing flow that references it from the cleanup.

Follow-up from #119399.

…e last place in the performance repo testing flow that references it from the cleanup.
Copy link
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

This PR completes the cleanup of HybridGlobalization references from the performance testing pipeline by removing the last remaining references in the iOS scenarios build jobs template. This follows up on the broader HybridGlobalization cleanup effort.

Key changes:

  • Removes the hybridGlobalization parameter from the template
  • Updates comments to remove HybridGlobalization references
  • Removes parameter passing to the build-perf-sample-apps template

@LoopedBard3 LoopedBard3 changed the title Remove HybridGlobalization from perf-ios-scenarios-build-jobs.yml, th… [PERF] Remove HybridGlobalization from perf-ios-scenarios-build-jobs.yml Oct 2, 2025
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @akoeplinger, @matouskozak, @simonrozsival
See info in area-owners.md if you want to be subscribed.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @akoeplinger, @matouskozak, @simonrozsival
See info in area-owners.md if you want to be subscribed.

@matouskozak matouskozak self-requested a review October 3, 2025 09:14
Copy link
Member

@matouskozak matouskozak left a comment

Choose a reason for hiding this comment

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

Thank you for the clean-up

@LoopedBard3 LoopedBard3 merged commit ae21049 into dotnet:main Oct 3, 2025
29 checks passed
@LoopedBard3 LoopedBard3 deleted the PerfRemoveLastiOSHGRemnants branch October 3, 2025 16:50
@github-actions github-actions bot locked and limited conversation to collaborators Nov 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants