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 the TargetFrameworks #459

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update the TargetFrameworks #459

wants to merge 1 commit into from

Conversation

Bond-009
Copy link
Member

Targeting netstandard2.0 and net6.0 should be enough for the library as that should support all currently supported verions of .Net Framework (4.6.2+) and .NET (6.0+) while allowing it to compile on linux.

(Technically .NET Framework 3.5 Service Pack 1 is still supported but we
currently don't support that anyway)

For the test projects I just went with the latest .NET version as .NET 5.0 isn't supported anymore.

Targeting netstandard2.0 and net6.0 should be enough for the library as
that should support all currently supported verions of .Net Framework
(4.6.2+) and .NET (6.0+) while allowing it to compile on linux.

(Technically .NET Framework 3.5 Service Pack 1 is still supported but we
 currently don't support that anyway)

For the test projects I just went with the latest .NET version as .NET
5.0 isn't supported anymore.
@cvium cvium added the blocked label Sep 29, 2023
@cvium
Copy link
Member

cvium commented Sep 29, 2023

We need to release 3.0.0 before this can go in.

@cvium cvium removed the blocked label Dec 7, 2023
Copy link

@Gauvino Gauvino left a comment

Choose a reason for hiding this comment

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

Maybe, updating the nuget package to the latest version is not a bad idea because of the change on dotnet 8

@@ -3,7 +3,7 @@
<Import Project="..\_Imports\Test.targets" />

<PropertyGroup>
<TargetFramework>net5.0</TargetFramework>
<TargetFramework>net7.0</TargetFramework>
Copy link

Choose a reason for hiding this comment

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

Suggested change
<TargetFramework>net7.0</TargetFramework>
<TargetFramework>net8.0</TargetFramework>

@@ -3,7 +3,7 @@
<Import Project="..\_Imports\Test.targets" />

<PropertyGroup>
<TargetFramework>net5.0</TargetFramework>
<TargetFramework>net7.0</TargetFramework>
Copy link

Choose a reason for hiding this comment

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

Suggested change
<TargetFramework>net7.0</TargetFramework>
<TargetFramework>net8.0</TargetFramework>

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

Successfully merging this pull request may close these issues.

3 participants