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

[UWP/WinUI3 Renderer] Consolidate Xaml IDLs #8334

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

krschau
Copy link
Collaborator

@krschau krschau commented Feb 28, 2023

Related Issue

#5888

Description

The ObjectModel and Rendering IDLs for UWP and WinUI3 are essentially the same, but were separated out into separate files. This change replaces these separate IDLs with shared IDLs in the Shared projects. Combining them and having one file that works for both types of Xaml will ensure that the two OMs and Renderers stay in sync.

How Verified

Build/pipelines.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service
Copy link
Contributor

Hi @krschau. This pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along.

1 similar comment
@microsoft-github-policy-service
Copy link
Contributor

Hi @krschau. This pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along.

@microsoft-github-policy-service
Copy link
Contributor

Staleness reset by krschau

@microsoft-github-policy-service
Copy link
Contributor

Hi @krschau. This pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along.

@microsoft-github-policy-service
Copy link
Contributor

Hi @krschau. This pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant