Skip to content

Remove Razor IVTs#43235

Merged
tmat merged 3 commits intodotnet:masterfrom
tmat:RazorIvtRemoval
Apr 23, 2020
Merged

Remove Razor IVTs#43235
tmat merged 3 commits intodotnet:masterfrom
tmat:RazorIvtRemoval

Conversation

@tmat
Copy link
Member

@tmat tmat commented Apr 9, 2020

Since dotnet/razor#1655 has been merged and inserted we can now constrain our IVTs further.

Fixes #35079

@tmat
Copy link
Member Author

tmat commented Apr 10, 2020

@NTaylorMullen

@tmat tmat marked this pull request as ready for review April 23, 2020 16:36
@tmat tmat requested review from a team as code owners April 23, 2020 16:36
@tmat tmat added the Area-IDE label Apr 23, 2020
@tmat tmat merged commit 88657fd into dotnet:master Apr 23, 2020
@ghost ghost added this to the Next milestone Apr 23, 2020
@tmat tmat deleted the RazorIvtRemoval branch April 23, 2020 18:28
@genlu
Copy link
Member

genlu commented Apr 24, 2020

This breaks ngen for Microsoft.VisualStudio.LanguageServices.Razor.dll
https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_git/VS/pullrequest/244527

genlu added a commit that referenced this pull request Apr 24, 2020
@sharwell sharwell modified the milestones: Next, temp, 16.7.P1 Apr 28, 2020
tmat added a commit to tmat/roslyn that referenced this pull request Jun 29, 2020
tmat added a commit that referenced this pull request Jul 1, 2020
* Revert "Revert "Remove Razor IVTs (#43235)""

This reverts commit 0dc4f3d.

* Remove Microsoft.VisualStudio.LanguageServices.Razor.RemoteClient.csproj

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate Razor features to the External Access pattern

5 participants