-
Notifications
You must be signed in to change notification settings - Fork 25.3k
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
Add preview 2 features #32034
Add preview 2 features #32034
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIT if you want to ... I'd recommend changing the broken sentence ("For example, the following code:" ... "Can now be simplified to the following code:") into two complete sentences. I suspect MT has a better shot if they're phrased as normal sentences.
Side note for us all (@Rick-Anderson, @wadepickett) ... but u guys may already recall it ... I was just reminded that under our new INCLUDE file scheme that updating an INCLUDE doesn't automatically update the Migration article. I'll probably flip the I made the Blazor Pre2 updates this morning (to the INCLUDE only). This PR will result in those changes being sucked into the article. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great with the update. Thanks Tom.
contributes to #31770
Internal previews