-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Fix error message/string(s) that reference project.json #5931
Comments
I think we should fix this (these) in the next patch. |
The specific error you listed we've fixed in dev - https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Razor/Internal/DefaultRoslynCompilationService.cs#L153. |
Oh right I remember this. Is it broken in 1.0.x and 1.1.x? |
It is. Note: It's generally less likely you'd run in to this error now since |
People starting with empty .NET Console app and then adding ASP.NET Core will hit this. We still do that in some demos, docs, etc. |
Ok, so putting this in 1.1.3 then. |
This patch fix is approved. Please follow the normal code review / pull request process and make sure you make the change in the correct branch. |
Close this bug? |
Preview builds of this patch fix should be available on the following feeds:
If you have a chance, please try out these preview builds and let us know if you have any feedback! |
Such as this: https://github.com/aspnet/Mvc/blob/rel/1.1.2/src/Microsoft.AspNetCore.Mvc.Razor/Internal/DefaultRoslynCompilationService.cs#L158
And we should scan the entire repo for any other references.
Does this seem worth porting to patch releases? How obscure is this?
The text was updated successfully, but these errors were encountered: