Skip to content

Conversation

davidwengier
Copy link
Member

Component Governance doesn't like that we have an old, vulnerable, commit from the VS Code repo referenced, so I thought I'd bring our TextMate grammars up to date.

We could also just dismiss the alerts since none of this is executable, but why not have the latest (answer: because sometimes VS doesn't like us updating TextMate grammars). Will reach out to the editor team and see if there is something they want us to do to validate first.

@davidwengier davidwengier requested a review from a team as a code owner October 13, 2025 21:18
Copy link
Member

@chsienki chsienki left a comment

Choose a reason for hiding this comment

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

Just for my own learning: what do we use these for?

@davidwengier
Copy link
Member Author

These are for classification of anything that doesn't support semantic tokens, which is to say Html, JavaScript and CSS. It's also the fallback for C# if there are things Roslyn doesn't give us tokens for (which I think there are).

@davidwengier
Copy link
Member Author

@davidwengier
Copy link
Member Author

@davidwengier
Copy link
Member Author

Manually tested with a very large .razor file, and cohsting/oop hit problems well before TextMate did.

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