Skip to content

Prepare an update for new C# changes#15

Draft
michaelblyons wants to merge 40 commits into
masterfrom
upstream/csharp/4204-master
Draft

Prepare an update for new C# changes#15
michaelblyons wants to merge 40 commits into
masterfrom
upstream/csharp/4204-master

Conversation

@michaelblyons
Copy link
Copy Markdown
Owner

@michaelblyons
Copy link
Copy Markdown
Owner Author

michaelblyons commented May 20, 2026

@keith-hall Was there a reason we had HTML comments instead of Razor comments as the marker in the syntax tests?

HTML comments in some spots seem to break downstream highlighting, and I'd like to know if that's a concern.

@do
<!-- HTML comment here detaches the block below from this `do` -->
{
    <p>forever</p>
} while(true)

Edit: Fixed.

@keith-hall
Copy link
Copy Markdown
Contributor

I don't remember there being a compelling reason for using HTML comments, I think it should be fine to switch

@michaelblyons michaelblyons force-pushed the upstream/csharp/4204-master branch from 2829c94 to efb51f8 Compare May 22, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants