Skip to content

Conversation

@jkoritzinsky
Copy link
Member

The Microsoft.NETCore.App package is being removed in dotnet/runtime#38457. The usage in ASP.NET Core can be replaced with the .NET Core Shared Framework VS insertion package and the Browser Debug Host transport package, which have the nonshipping version number that matches a runtime build.

…th other non-shipping packages from dotnet/runtime to get a non-shipping version.
@dougbu dougbu requested a review from a team November 11, 2020 19:28
@dougbu
Copy link
Contributor

dougbu commented Nov 11, 2020

Adding @dotnet/aspnet-build for awareness.

Copy link
Contributor

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

Thanks❕

@mkArtakMSFT mkArtakMSFT added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Nov 11, 2020
@jkoritzinsky
Copy link
Member Author

@dougbu can you triage the failing tests for me? I don't know enough about the aspnetcore repo.

@dougbu
Copy link
Contributor

dougbu commented Nov 11, 2020

@jkoritzinsky I didn't look at all failures but the only problems I saw were instances of https://github.com/dotnet/core-eng/issues/11391. I told AzDO to rerun the failed jobs.

- adjust version properties to align with 44c0e66 / dotnet#27784
@dougbu
Copy link
Contributor

dougbu commented Nov 16, 2020

Latest failures are more of dotnet/runtime#44700. Good news is flaky AzDO feeds didn't break all of the jobs I retried last time. May slowly get to success.

@dougbu dougbu merged commit 634b50e into dotnet:master Nov 17, 2020
@dougbu
Copy link
Contributor

dougbu commented Nov 17, 2020

Thanks very much @jkoritzinsky

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

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants