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

Use explicit platform versions #23995

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Use explicit platform versions #23995

merged 1 commit into from
Aug 5, 2024

Conversation

mattleibow
Copy link
Member

@mattleibow mattleibow commented Aug 3, 2024

Description of Change

This is used to avoid the case where some SDKs bump their platform version and we end up forcing the entire ecosystem to update SDKs.

This just updates the library projects to use explicit versions, all the sample and tests apps still use the unversioned TFMs like a typical dev would.

This is an alternative to #23966 but makes sure to use the old versions for net7 and the new versions for net8.

This is used to avoid the case where some SDKs bump their platform
version and we end up forcing the entire ecosystem to update SDKs.
Comment on lines +136 to +144
<!-- Previous .NET -->
<IosPreviousTargetFrameworkVersion>16.1</IosPreviousTargetFrameworkVersion>
<TvosPreviousTargetFrameworkVersion>16.1</TvosPreviousTargetFrameworkVersion>
<MacCatalystPreviousTargetFrameworkVersion>16.1</MacCatalystPreviousTargetFrameworkVersion>
<MacosPreviousTargetFrameworkVersion>13.0</MacosPreviousTargetFrameworkVersion>
<AndroidPreviousTargetFrameworkVersion>33.0</AndroidPreviousTargetFrameworkVersion>
<WindowsPreviousTargetFrameworkVersion>10.0.19041.0</WindowsPreviousTargetFrameworkVersion>
<Windows2PreviousTargetFrameworkVersion>10.0.20348.0</Windows2PreviousTargetFrameworkVersion>
<TizenPreviousTargetFrameworkVersion>7.0</TizenPreviousTargetFrameworkVersion>
Copy link
Member

Choose a reason for hiding this comment

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

Are there some projects that use .NET 7? Can we get rid of those, and only target the current supported version? 7 support ended in May.

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure, but I think that would be a breaking change? The current nugets have .NET 7 so I am not sure if we can remove net7.

Copy link
Member Author

Choose a reason for hiding this comment

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

Seems we still have a lot of people using .net 7 since it still is in VS

@PureWeen PureWeen added this to the .NET 8 SR8 milestone Aug 5, 2024
@PureWeen PureWeen merged commit 946f462 into main Aug 5, 2024
97 checks passed
@PureWeen PureWeen deleted the dev/platform-versions-v2 branch August 5, 2024 17:21
@github-actions github-actions bot locked and limited conversation to collaborators Sep 8, 2024
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.

4 participants