feat: added support for ignoring using directives

Author: TimothyMakkison

Src/CSharpier.Core/CSharp/SyntaxPrinter/UsingDirectives.cs

@@ -68,6 +68,11 @@ or SyntaxKind.UndefDirectiveTrivia
             }
         }
 
+        if (Token.HasLeadingCommentMatching(usings[0], CSharpierIgnore.IgnoreStartRegex))
+        {
+            initialComments.Clear();
+        }
+
         docs.Add(Token.PrintLeadingTrivia([.. initialComments], context));
         if (keepUsingsUntilEndIf)
         {
@@ -94,6 +99,7 @@ or SyntaxKind.UndefDirectiveTrivia
         }
 
         var isFirst = true;
+        var prevShouldFormat = true;
         var index = 0;
         var reorderedDirectives = false;
         foreach (var groupOfUsingData in GroupUsings(usingList, [.. triviaWithinIf], context))
@@ -102,10 +108,15 @@ or SyntaxKind.UndefDirectiveTrivia
             {
                 if (!isFirst)
                 {
-                    docs.Add(Doc.HardLine);
+                    if (usingData.ShouldFormat || prevShouldFormat)
+                    {
+                        docs.Add(Doc.HardLine);
+                    }
                 }
 
-                if (usingData.LeadingTrivia != Doc.Null)
+                prevShouldFormat = usingData.ShouldFormat;
+
+                if (usingData.LeadingTrivia != Doc.Null && usingData.ShouldFormat)
                 {
                     docs.Add(usingData.LeadingTrivia);
                 }
@@ -117,7 +128,9 @@ or SyntaxKind.UndefDirectiveTrivia
                     }
 
                     index++;
-                    docs.Add(UsingDirective.Print(usingData.Using, context, printExtraLines));
+                    docs.Add(usingData.ShouldFormat
+                        ? UsingDirective.Print(usingData.Using, context, printExtraLines)
+                        : CSharpierIgnore.PrintWithoutFormatting(usingData.Using, context));
                 }
 
                 isFirst = false;
@@ -159,10 +172,21 @@ PrintingContext context
         var directiveGroup = new List<UsingData>();
         var ifCount = 0;
         var isFirst = true;
-
+        var shouldFormat = true;
+        
         foreach (var usingDirective in usings)
         {
             var openIf = ifCount > 0;
+            if (Token.HasLeadingCommentMatching(usingDirective, CSharpierIgnore.IgnoreEndRegex))
+            {
+                openIf = true;
+                shouldFormat = true;
+            }
+            else  if (Token.HasLeadingCommentMatching(usingDirective, CSharpierIgnore.IgnoreStartRegex))
+            {
+                shouldFormat = false;
+            }
+            
             foreach (var directive in usingDirective.GetLeadingTrivia().Where(o => o.IsDirective))
             {
                 if (directive.RawSyntaxKind() is SyntaxKind.IfDirectiveTrivia)
@@ -182,6 +206,7 @@ PrintingContext context
                     {
                         Using = usingDirective,
                         LeadingTrivia = PrintLeadingTrivia(usingDirective),
+                        ShouldFormat = shouldFormat,
                     }
                 );
             }
@@ -190,17 +215,22 @@ PrintingContext context
                 if (openIf)
                 {
                     directiveGroup.Add(
-                        new UsingData { LeadingTrivia = PrintLeadingTrivia(usingDirective) }
+                        new UsingData { LeadingTrivia = PrintLeadingTrivia(usingDirective), ShouldFormat = shouldFormat}
                     );
                 }
 
                 var usingData = new UsingData
                 {
                     Using = usingDirective,
                     LeadingTrivia = !openIf ? PrintLeadingTrivia(usingDirective) : Doc.Null,
+                    ShouldFormat = shouldFormat,
                 };
 
-                if (usingDirective.GlobalKeyword.RawSyntaxKind() != SyntaxKind.None)
+                if (!shouldFormat)
+                {
+                    directiveGroup.Add(usingData);
+                }
+                else if (usingDirective.GlobalKeyword.RawSyntaxKind() != SyntaxKind.None)
                 {
                     if (usingDirective.Alias is not null)
                     {
@@ -274,6 +304,7 @@ private class UsingData
     {
         public Doc LeadingTrivia { get; init; } = Doc.Null;
         public UsingDirectiveSyntax? Using { get; init; }
+        public required bool ShouldFormat { get; init; } = true;
     }
 
     private static bool IsSystemName(NameSyntax value)

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore1.expected.test

@@ -0,0 +1,7 @@
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore1.test

@@ -0,0 +1,7 @@
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore2.expected.test

@@ -0,0 +1,10 @@
+using C;
+using D;
+
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore2.test

@@ -0,0 +1,10 @@
+using D;
+using C;
+
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore3.expected.test

@@ -0,0 +1,9 @@
+using C;
+using D;
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore3.test

@@ -0,0 +1,9 @@
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end
+using D;
+using C;

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore4.expected.test

@@ -0,0 +1,21 @@
+using C;
+using D;
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end
+
+
+
+
+
+// csharpier-ignore-start
+
+using F;
+
+using E;
+
+// csharpier-ignore-end

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore4.test

@@ -0,0 +1,21 @@
+// csharpier-ignore-start
+
+using B;
+
+using A;
+
+// csharpier-ignore-end
+using D;
+using C;
+
+
+
+
+// csharpier-ignore-start
+
+using F;
+
+using E;
+
+// csharpier-ignore-end
+

Src/CSharpier.Tests/FormattingTests/TestFiles/cs/UsingDirectives_CSharpierIgnore5.expected.test

@@ -0,0 +1,18 @@
+using C;
+using D;
+// csharpier-ignore-start
+#if BUILD_MSI_TASKS
+
+
+using System;
+using Microsoft.
+Build.Framework;
+
+
+#endif
+
+// csharpier-ignore-end
+
+namespace RepoTasks;
+
+class ClassName { }