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

feat(csharp): upgrade to netcoreapp3.1 #1241

Merged
merged 3 commits into from
Feb 7, 2020

Conversation

RomainMuller
Copy link
Contributor

@RomainMuller RomainMuller commented Feb 6, 2020

From netcoreap3.0, in order to reflect the current
recommended framework version for .NET 3.x.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Commit Message

feat(csharp): upgrade to netcoreapp3.1 (#1241)

As netcoreapp3.0 is going to be EOL pretty soon (early March 2020),
upgrading our target framework to netcoreapp3.1 early in order to
catch any surprises. Since netcoreapp3.1 is nothing but an LTS version
of netcoreapp3.0 (plus some bug fixes), it should be transparent to users
who may only need to upgrade their .NET SDK.

From `netcoreap3.0`, in order to reflect the current
recommended framework version for .NET 3.x.
@RomainMuller RomainMuller requested review from MrArnoldPalmer and a team as code owners February 6, 2020 16:56
@RomainMuller RomainMuller self-assigned this Feb 6, 2020
@RomainMuller RomainMuller added effort/small Small work item – less than a day of effort language/dotnet Related to .NET bindings (C#, F#, ...) labels Feb 6, 2020
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@rix0rrr
Copy link
Contributor

rix0rrr commented Feb 7, 2020

What is the impact of this? Specifically, what are the benefits and what are the costs?

  • Are people using .NET core 3.1 currently unable to use jsii because we're targeting 3.0?
  • Do people using .NET core 3 now have to upgrade to use jsii after this change?
  • What are the advantages of upping our requirements?
  • Does .NET core 3.1 have new features we are planning to take advantage of?
  • Is .NET core 3.0 end of life?

To me, it seems nice to our customers to have as broad compatibility as possible for our software, to not force them to upgrade unnecessarily. It's why we target Java 8 (instead of 13) and NodeJS 10 (instead of 13) and Python 3.6 (instead of 3.7).

@RomainMuller
Copy link
Contributor Author

RomainMuller commented Feb 7, 2020

@rix0rrr

  • netcoreapp3.0 can be built by .NET Core 3.1 SDKs
  • Yes, people who have not yet upgraded to the current release of the dotnet-sdk will need to do so
  • There is no advantage beyond the fact that netcoreapp3.0 is soon to become EOL.
  • There's no new features we plan on using
  • netcoreapp3.0is going EOL on March 3, 2020. I'd rather upgrade early and be able to revert while we work out wrinkles than do it after EOL. netcoreapp3.1 is the LTS release.
    • The upgrade was recommended by our .NET SDK engineers, who strongly advised we upgrade our target framework to 3.1 on the first occasion (it was not released yet when we moved to netcoreapp3.0).

Ref: https://dotnet.microsoft.com/platform/support/policy/dotnet-core

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Copy link
Contributor

@rix0rrr rix0rrr left a comment

Choose a reason for hiding this comment

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

Thanks for the info!

@RomainMuller
Copy link
Contributor Author

(Note that this code requires a new Superchain image to be live before it'll pass PR validations)

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@RomainMuller RomainMuller removed the request for review from MrArnoldPalmer February 7, 2020 16:22
@RomainMuller RomainMuller merged commit c324439 into master Feb 7, 2020
@RomainMuller RomainMuller deleted the rmuller/updade-dotnet-3.1 branch February 7, 2020 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort/small Small work item – less than a day of effort language/dotnet Related to .NET bindings (C#, F#, ...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants