-
Notifications
You must be signed in to change notification settings - Fork 79
React to breaking changes in Microsoft.AspNetCore.App #836
Conversation
* Add packages references for EF Core, when necessary * Add packages references for SpaServices to Spa templates * Remove PackageReference to Microsoft.AspNetCore.App altogether
src/Microsoft.DotNet.Web.ProjectTemplates/RazorClassLibrary-CSharp.csproj.in
Show resolved
Hide resolved
src/Microsoft.DotNet.Web.ProjectTemplates/RazorClassLibrary-CSharp.csproj.in
Show resolved
Hide resolved
cc @ajcvickers you may be interested too, since most of this was about adding PackageRefs for EFCore |
🆙 📅 tests are passing with the latest build of aspnetcore. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume you have a good answer for my question, I'd just like to hear it.
@@ -12,7 +12,7 @@ | |||
</PropertyGroup> | |||
|
|||
<ItemGroup> | |||
<PackageReference Include="Microsoft.AspNetCore.App" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess I'm confused about under what circumstances we need a FrameworkReference? Why do we need it in RazorClassLibrary but not here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current plan of record is that Microsoft.NET.Sdk.Web
implies a FrameworkReference to aspnetcore. RazorClassLibrary doesn't use the websdk, it uses Microsoft.NET.Sdk.Razor
WIP until dotnet/aspnetcore#4004 merges and has a build available.Changes:
Resolves dotnet/aspnetcore#3612
cc @dsplaisted