Update repository and path for Razor repo#313011
Update repository and path for Razor repo#313011davidwengier wants to merge 1 commit intomicrosoft:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Razor extension’s grammar update script to pull the cshtml TextMate grammar from the Razor sources after they were merged into the dotnet/roslyn repository.
Changes:
- Switch the upstream grammar repository from
dotnet/razortodotnet/roslyn. - Update the upstream path to the
aspnetcorerazor.tmLanguage.jsonfile within the Roslyn repo layout.
| const razorGrammarRepo = 'dotnet/roslyn'; | ||
| const grammarPath = 'src/Razor/src/Razor/src/Microsoft.VisualStudio.RazorExtension/EmbeddedGrammars/aspnetcorerazor.tmLanguage.json'; | ||
| vscodeGrammarUpdater.update(razorGrammarRepo, grammarPath, './syntaxes/cshtml.tmLanguage.json', grammar => patchGrammar(grammar), 'main'); |
There was a problem hiding this comment.
This switches the upstream grammar source to dotnet/roslyn, but the Razor extension’s tracked upstream metadata is still pointing at dotnet/razor (e.g. extensions/razor/cgmanifest.json and the information_for_contributors/version fields in syntaxes/cshtml.tmLanguage.json). To keep attribution/compliance and contributor guidance accurate, please also update the cgmanifest entry to dotnet/roslyn and refresh the generated grammar file (typically by running this updater and committing the updated cshtml.tmLanguage.json with the new upstream commit link).
We merged the Razor repo into Roslyn so the source for the cshtml tmLanguage needs to change.