Skip to content

Conversation

@davidwengier
Copy link
Member

No description provided.

@davidwengier davidwengier requested review from a team and Copilot December 16, 2025 23:12
@davidwengier davidwengier requested a review from a team as a code owner December 16, 2025 23:12
@github-actions github-actions bot added the Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch label Dec 16, 2025
@dotnet-policy-service
Copy link
Contributor

Thanks for your PR, @@davidwengier.
To learn about the PR process and branching schedule of this repo, please take a look at the SDK PR Guide.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for RazorLangVersion 11.0 for .NET 11.0 by introducing a new version-specific configuration block in the Razor SDK targets file.

Key Changes:

  • Adds a new <When> condition block for .NETCoreApp 11.0+ that sets RazorLangVersion to 11.0
  • Sets all necessary version flags (_TargetingNET110OrLater and earlier version flags)
  • Enables UseRazorSourceGenerator and _RazorUseRoslynTokenizer features for the new version

@davidwengier
Copy link
Member Author

Are these sorts of PRs better done in the VMR these days, so we can be sure things are inserted in the right places/order first?

<_TargetingNET100OrLater>true</_TargetingNET100OrLater>
<UseRazorSourceGenerator Condition="'$(Language)' == 'C#' AND '$(UseRazorSourceGenerator)' == '' ">true</UseRazorSourceGenerator>
<RazorLangVersion Condition="'$(RazorLangVersion)' == '' ">9.0</RazorLangVersion>
<RazorLangVersion Condition="'$(RazorLangVersion)' == '' ">10.0</RazorLangVersion>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed with the team and we're going to backport this to 10.0.2xx too

@davidwengier
Copy link
Member Author

That answers my question: Need to wait for Razor to flow in here, via VMR, before this can be merged. Hopefully that is all in place to succeed in time...

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

Labels

Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants