Skip to content

Commit 29dad74

Browse files
Apply suggestions from code review
Co-authored-by: Copilot <[email protected]>
1 parent 96a10f3 commit 29dad74

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

src/Razor/src/Microsoft.CodeAnalysis.Razor.Workspaces/Formatting/Passes/HtmlFormattingPass.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ private async Task<ImmutableArray<TextChange>> FilterIncomingChangesAsync(Format
8181
// The Html formatter in VS Code wraps long lines, based on a user setting, but when there
8282
// are long C# string literals that ends up breaking the code. For example:
8383
//
84-
// @("this is a long string that spans past some user set maximmum limit")
84+
// @("this is a long string that spans past some user set maximum limit")
8585
//
8686
// could become
8787
//
@@ -91,7 +91,7 @@ private async Task<ImmutableArray<TextChange>> FilterIncomingChangesAsync(Format
9191
// That doesn't compile, and depending on the scenario, can even cause a crash inside the
9292
// Roslyn formatter.
9393
//
94-
// Strictly speaking if literal is a verbatim string, or multline raw string literal, then
94+
// Strictly speaking if literal is a verbatim string, or multiline raw string literal, then
9595
// it would compile, but it would also change the value of the string, and since these edits
9696
// come from the Html formatter which clearly has no idea it's doing that, it is safer to
9797
// disregard them all equally, and let the user make the final decision.

src/Razor/test/Microsoft.VisualStudio.LanguageServices.Razor.Test/Cohost/Formatting/HtmlFormattingPassTest.cs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,6 @@
2323

2424
namespace Microsoft.VisualStudio.LanguageServices.Razor.Test.Cohost.Formatting;
2525

26-
/// <summary>
27-
/// Not tests of the formatting log, but tests that use formatting logs sent in
28-
/// by users reporting issues.
29-
/// </summary>
3026
[Collection(HtmlFormattingCollection.Name)]
3127
public class HtmlFormattingPassTest(FormattingTestContext context, HtmlFormattingFixture fixture, ITestOutputHelper testOutput)
3228
: FormattingTestBase(context, fixture.Service, testOutput), IClassFixture<FormattingTestContext>

0 commit comments

Comments
 (0)