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

Update : Remove Requirements for the Latest WinUI SDK in Maui #3580

Merged
merged 2 commits into from Jul 9, 2023

Conversation

ChrisPulman
Copy link
Member

What kind of change does this PR introduce?

Update

What is the current behaviour?

There is a possibility that we could use functionality that is not available on the end users target by importing a newer SDK than the target we use.

What is the new behaviour?

By removing the latest SDK we will automatically fall back to our target framework 19041 this will ensure that we don't use components that are not compatible with older versions of windows 10 whilst enabling the end user a simple means of using the library with the SDK automatically being by the IDE selecting the correct SDK based upon the end users Target choices

What might this PR break?

None expected, those who have already added the newer SDK's can continue to do so

Please check if the PR fulfils these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Other information:

By removing the latest SDK we will automatically fall back to our target framework 19041 this will ensure that we dont use components that are not compatible with older versions of windows 10 whilst enabling the end user a simple means of using the library with the SDK automatically being by the IDE selecting the correct SDK based upon the end users Target choices
@ChrisPulman ChrisPulman changed the title Update : Remove Requirements for the Latest SDK Update : Remove Requirements for the Latest WinUI SDK in Maui Jul 9, 2023
@codecov
Copy link

codecov bot commented Jul 9, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (3467c36) 64.07% compared to head (28def40) 64.07%.

❗ Current head 28def40 differs from pull request most recent head 512dcb6. Consider uploading reports for the commit 512dcb6 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3580   +/-   ##
=======================================
  Coverage   64.07%   64.07%           
=======================================
  Files         157      157           
  Lines        5762     5762           
=======================================
  Hits         3692     3692           
  Misses       2070     2070           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@glennawatson glennawatson merged commit 6d309b2 into main Jul 9, 2023
1 check passed
@glennawatson glennawatson deleted the CP_RemoveRequirementOnLatestSDK branch July 9, 2023 23:18
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 24, 2023
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.

None yet

2 participants