-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[main] Update dependencies from dotnet/razor #31508
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
[main] Update dependencies from dotnet/razor #31508
Conversation
…28.7 Microsoft.AspNetCore.Mvc.Razor.Extensions.Tooling.Internal , Microsoft.CodeAnalysis.Razor.Tooling.Internal , Microsoft.NET.Sdk.Razor.SourceGenerators.Transport From Version 7.0.0-preview.23167.3 -> To Version 7.0.0-preview.23178.7
|
Lots 'O test failures here, may you take a peek my friends @dotnet/razor-tooling ? |
|
@dotnet/razor-compiler did you want to have a first look? |
|
Interesting... had no idea what razor-tooling is. but now I know. 😉 |
|
I admit, I am also confused and surprised by it (such that I edited my comment 😝). |
4e8cea8 to
08b652f
Compare
|
The error is Looks like the way Guessing something like this could fix the problem: dotnet/razor#8542 I also attempted to fix it here by adding
|
08b652f to
f413eae
Compare
|
@jjonescz Any updates here? Thanks for taking a look |
The PR dotnet/razor#8542 in Razor which should fix this has been merged some time ago. I'm not sure when it'll flow here. |
|
is the razor sdk flowing properly?cc @mmitche |
|
Perhaps the bot didn't update since I did a force push on this PR before. So I bumped the versions manually. |
This pull request updates the following dependencies
From https://github.com/dotnet/razor