dotnet / razor

Compiler and tooling experience for Razor ASP.NET Core apps in Visual Studio, Visual Studio for Mac, and VS Code.
https://asp.net
MIT License
506 stars 196 forks source link

Consolidate Razor compiler code into a single DLL #8400

Closed jjonescz closed 9 months ago

jjonescz commented 1 year ago

Merge the following DLLs into one (Microsoft.CodeAnalysis.Razor.Compiler):

Merge tests into one project, as well (Microsoft.CodeAnalysis.Razor.Compiler.Tests).

Roadmap:

Old plan: - [x] https://github.com/dotnet/razor/pull/8386 - [x] https://github.com/dotnet/razor/pull/8395 - [x] https://github.com/dotnet/razor/pull/8471 - [ ] Update VSIX on Roslyn side similarly to https://github.com/dotnet/roslyn/pull/67501 - [ ] Update transports. - [ ] Update runtime compilation in aspnetcore. - [ ] Delete the old packages.
To allow servicing of the old compiler (branch `release/dev17.6`): Not needed, I think, since we have shims now (#8941) - [ ] https://github.com/dotnet/razor/pull/8594 - [ ] Take dependency on the old compiler in the main branch. > **Note** To service the old compiler: update the 17.6 branch, release new version of this old package, update main branch to consume that version of the old package, build and insert into VS.

See also:

jaredpar commented 1 year ago

Integration tests are blocked on https://github.com/dotnet/dnceng/issues/563