Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MUX Experimental: Remove hard-coded package version from GetLocalizedStringResourceFromWinUI #6265

Merged
merged 2 commits into from
Nov 10, 2021

Conversation

kmahone
Copy link
Member

@kmahone kmahone commented Nov 9, 2021

This function was using a hardcoded string to look up resources from Microsoft.UI.Xaml.2.6 framework package. This means that Microsoft.Experimental.UI.Xaml nuget package could only be used with WinUI 2.6 release. This PR updates the code to find any Microsoft.UI.Xaml.* framework package to use for finding localized resources.

This also ensures that Microsoft.Experimental.UI.Xaml.pdb debugging symbols get published to the drop in the Pipeline.

Closes #6242

@kmahone kmahone requested a review from teaP November 9, 2021 20:05
@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Nov 9, 2021
@kmahone
Copy link
Member Author

kmahone commented Nov 9, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@teaP teaP merged commit ca0bc63 into main Nov 10, 2021
@teaP teaP deleted the user/kmahone/winuilabsfixes branch November 10, 2021 18:55
@StephenLPeters StephenLPeters added area-DevInternal Internal build infrastructure, code cleanup, engineering efficiency team-Controls Issue for the Controls team and removed needs-triage Issue needs to be triaged by the area owners labels Feb 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-DevInternal Internal build infrastructure, code cleanup, engineering efficiency team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MUX Experimental: ResourceAccessor::GetLocalizedStringResourceFromWinUI has hardcoded string for WinUI 2.6
3 participants