Skip to content

Updates NetCoreApp 3.0 builds to NetCoreApp 3.1 #4888

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

Merged
merged 7 commits into from
Mar 23, 2020

Conversation

mstfbl
Copy link
Contributor

@mstfbl mstfbl commented Feb 25, 2020

Fix #4865 . Updates our builds using NetCoreApp 3.0 to NetCoreApp 3.1.

@mstfbl mstfbl force-pushed the netcoreapp3.0to3.1 branch from dd5ec10 to ba1757b Compare March 10, 2020 18:59
@mstfbl mstfbl marked this pull request as ready for review March 19, 2020 20:09
@mstfbl mstfbl requested a review from a team as a code owner March 19, 2020 20:09
@@ -13,20 +13,20 @@ resources:
jobs:
- template: /build/ci/job-template.yml
parameters:
name: Centos_x64_NetCoreApp30
name: Centos_x64_NetCoreApp31
Copy link
Contributor

@harishsk harishsk Mar 23, 2020

Choose a reason for hiding this comment

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

I don't think we will support multiple versions of .netcoreapp 3.x.
Can we simply make the whole thing NetCoreApp3?

Copy link
Contributor Author

@mstfbl mstfbl Mar 23, 2020

Choose a reason for hiding this comment

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

That makes sense, though I do think that specifying which version of NetCoreApp3 (3.0 or 3.1) we're using would be helpful if in the future we see bugs that are specific to 3.0 or 3.1, as you pointed out with respect to #4933 .

I will update these references to version 3.0/3.1 to only have version "3", if you do believe that specifying the exact iteration of NetCoreApp3 won't be helpful as I mentioned above.

@harishsk
Copy link
Contributor

@mstfbl It appears that upgrading to NetCoreApp31 will fix #4933
Can you please address the comment above and verify if it really fixes that issue and close both?

Copy link
Contributor

@harishsk harishsk left a comment

Choose a reason for hiding this comment

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

:shipit:

@mstfbl mstfbl merged commit 94af826 into dotnet:master Mar 23, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Mar 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

move from net core 3.0 to 3.1 on CI test
3 participants