Skip to content

Conversation

@StephaneDelcroix
Copy link
Contributor

Description of Change

fix resources localization

Copilot AI review requested due to automatic review settings September 3, 2025 12:04
@StephaneDelcroix StephaneDelcroix requested a review from a team as a code owner September 3, 2025 12:04

This comment was marked as outdated.

simonrozsival
simonrozsival previously approved these changes Sep 3, 2025
mattleibow
mattleibow previously approved these changes Sep 3, 2025
rolfbjarne
rolfbjarne previously approved these changes Sep 3, 2025
Copy link
Member

@jfversluis jfversluis left a comment

Choose a reason for hiding this comment

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

Build error...?

.dotnet/sdk/10.0.100-rc.1.25416.112/Microsoft.Common.CurrentVersion.targets(3442,5): Error MSB3552: Resource file "../../../Controls/src/SourceGen/xlf/MauiGResources.resx" cannot be found.

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 fixes a resources localization issue by correcting file paths in the localization configuration. The changes ensure that localized resource files are properly generated and referenced in the correct directory structure.

Key changes:

  • Updates the source file path for MauiGResources.resx in the unit test project to remove incorrect "xlf" subdirectory
  • Corrects the output path in the localization configuration to include the "xlf" subdirectory for proper localized file generation

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/Core/tests/UnitTests/Core.UnitTests.csproj Fixes the source path reference for MauiGResources.resx by removing the incorrect "xlf" directory from the Include path
eng/automation/LocProject.json Corrects the OutputPath to include "xlf\" subdirectory for proper localized resource file generation

@jfversluis jfversluis enabled auto-merge September 4, 2025 09:00
@jfversluis jfversluis disabled auto-merge September 4, 2025 09:38
@jfversluis jfversluis merged commit 4c11583 into net10.0 Sep 4, 2025
146 of 148 checks passed
@jfversluis jfversluis deleted the dev/stdelc/xsg/fixl10n branch September 4, 2025 09:38
@github-actions github-actions bot locked and limited conversation to collaborators Oct 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants