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 projets to .netstandart2.1 and .netcoreapp3.1 (Signed-off-by: H de Labareyre <henry.delabareyre@7opteam.com>) #59

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

Conversation

henryDeL49
Copy link

Just update framework target as these .net libraries are about to be not supported.

@Namaneo
Copy link
Contributor

Namaneo commented Aug 11, 2021

Hello,

According to this documentation, .NET Framework is far from being deprecated. Updating to netstandard2.1 would entirely drop its support, and this is not something we want. That said, netcoreapp2.0 is indeed deprecated, and upgrading the test project to netcoreapp3.1 makes sense.

Regarding the .NET Standard version, were you motivated by a specific need? As far as I'm aware of, any project that is compatible with netstandard2.1 will be compatible with netstandard2.0, but there might be a case I don't see. 🙂

EDIT: If you wish to update the pull-request according to what has been said above, please take a look at the CONTRIBUTING section, especially the Signed-off-by part.

@henryDeL49 henryDeL49 changed the title - update projets to .netstandart2.1 and .netcoreapp3.1 - update projets to .netstandart2.1 and .netcoreapp3.1 Signed-off-by: Random J Developer <random@developer.org> Aug 13, 2021
@henryDeL49 henryDeL49 changed the title - update projets to .netstandart2.1 and .netcoreapp3.1 Signed-off-by: Random J Developer <random@developer.org> - update projets to .netstandart2.1 and .netcoreapp3.1 (Signed-off-by: H de Labareyre <henry.delabareyre@7opteam.com>) Aug 13, 2021
@henryDeL49
Copy link
Author

Hi,

Indeed thank you for the precision!
.netcoreapp2.0 is the framework that is deprecated. If we wish to update the test project to netcoreapp3.1, we must update the library to netstandard2.1according to this documentation.

I don't see any regression using .netstandard2.1, and I have executed unit tests.

What motivated me to do this were 2 reason :

  • Using the project for a solution made up of other netcoreapp3.1 projects
  • Execute the unit tests on VS 2019 as test execution did not work when the project was compiled with netcoreapp2.0

Ok for the sign-off part, I've modified the pull request. Do you wish to have it in the commit comment or in the pull request title ?

Regards,

@Namaneo
Copy link
Contributor

Namaneo commented Aug 13, 2021

The documentation you linked lists the minimum compatible versions:

  • netstandard2.0 is compatible with all versions of .NET Core after netcoreapp2.0
  • netstandard2.1 is compatible with all versions of .NET Core after netcoreapp3.0

Which is equivalent to:

  • netcoreapp2.0 is only compatible with netstandard2.0
  • netcoreapp3.0 is compatible with both netstandard2.0 and netstandard2.1

In your situation, you should be able to use this library on a netcoreapp3.1. 🙂 have you experienced issues installing the NuGet package?

Side note: the Signed-off-by must be on the commit, not in the issue title. 😉Something like that:

Your commit message here

Signed-off-by: Your Name <your@email.address>

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.

2 participants