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

Migrate to .net core 3.1 #277

Closed
ChrisKujawa opened this issue May 10, 2021 · 9 comments · Fixed by #328
Closed

Migrate to .net core 3.1 #277

ChrisKujawa opened this issue May 10, 2021 · 9 comments · Fixed by #328

Comments

@ChrisKujawa
Copy link
Collaborator

Migrating to .net core 3.1 gives us some new possibilities like:

Furthermore .net core 2.1 is as LTS only supported until August 21, 2021, see https://dotnet.microsoft.com/platform/support/policy/dotnet-core

@Christian-Oleson
Copy link
Contributor

Christian-Oleson commented Oct 1, 2021

@Zelldon
.NET Core 3.1 is only supported through December 3, 2022. Are you interested in moving from .NET 2.1 to .NET 5? I realize that .NET 5 also is EOL in 2022, but .NET 6 would be very easy to update to, and that would have a very long support life.

@Christian-Oleson
Copy link
Contributor

However, regardless, I'll gladly take this one.

@Christian-Oleson
Copy link
Contributor

The other question I have is regarding the client project. This one is in .NET Standard. Is there a reason we are targeting standard (E.g. backward compatibility)? If so, I'll leave it alone, if not, I'll address the issue.

@ChrisKujawa
Copy link
Collaborator Author

Hey @Christian-Oleson

thanks for stepping in, to answer your question:

.NET Core 3.1 is only supported through December 3, 2022. Are you interested in moving from .NET 2.1 to .NET 5? I realize that .NET 5 also is EOL in 2022, but .NET 6 would be very easy to update to, and that would have a very long support life.

I would prefer to go with the LTS versions, we can update this then next year again.

The other question I have is regarding the client project. This one is in .NET Standard. Is there a reason we are targeting standard (E.g. backward compatibility)? If so, I'll leave it alone, if not, I'll address the issue.

Yes since it should be used in other libs, no matter of windows, linux etc. As far as I know it would be the best to use .net standard here to have a broader audience and support more versions, but as mentioned .net core 2.1 is already out of life, so I would say we can update to .net standard 2.1 (which is then implemented by .net core 3.x afaik). What do you think? Correct me please if I'm wrong I'm not a .net native 😅

@Christian-Oleson
Copy link
Contributor

Hey @Christian-Oleson

thanks for stepping in, to answer your question:

.NET Core 3.1 is only supported through December 3, 2022. Are you interested in moving from .NET 2.1 to .NET 5? I realize that .NET 5 also is EOL in 2022, but .NET 6 would be very easy to update to, and that would have a very long support life.

I would prefer to go with the LTS versions, we can update this then next year again.

I totally understand. .NET 6 comes out in November, and this should be a LTS version. If you want, I can take on that change if you want (confirm this by) opening the .NET 6 migration issue.

The other question I have is regarding the client project. This one is in .NET Standard. Is there a reason we are targeting standard (E.g. backward compatibility)? If so, I'll leave it alone, if not, I'll address the issue.

Yes since it should be used in other libs, no matter of windows, Linux, etc. As far as I know, it would be best to use .NET standard here to have a broader audience and support more versions, but as mentioned .net core 2.1 is already out of life, so I would say we can update to .NET standard 2.1 (which is then implemented by .net core 3.x afaik). What do you think? Correct me please if I'm wrong I'm not a .net native 😅

.NET Core and .NET 5 are cross-platform, including Windows, Linux, Mac, etc. If that is the concern, updating this to .NET Core isn't an issue. I wouldn't do this yet, because in .NET Core 3.1 it is still technically valid to reference .NET standard. However, in .NET 5, that replaces standard.

The only time you DONT want to change this is when you end up needed the library to reference .NET Framework. And then, depending on what version of .NET Framework you are referencing, you won't want to update to .NET Standard 2.1. Please let me know if you're desiring to use .NET Framework. 😄

ChrisKujawa added a commit that referenced this issue Oct 5, 2021
@ChrisKujawa ChrisKujawa reopened this Oct 27, 2021
@ChrisKujawa
Copy link
Collaborator Author

In order to migrate to grpc-dotnet we probably need to migrate the client project

@Christian-Oleson
Copy link
Contributor

In order to migrate to grpc-dotnet we probably need to migrate the client project

Can you confirm further what you mean @Zelldon ? Being less familiar with the project, I'm not fully following

@ChrisKujawa
Copy link
Collaborator Author

Sorry I meant this #348 in order to migrate to grpc-dotnet we need to upgrade the client project to .net core 3.x as far as I understamd

@ChrisKujawa
Copy link
Collaborator Author

With #543 we migrated to .net 7.0

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

Successfully merging a pull request may close this issue.

2 participants