Improve warning messages on invalid CODEOWNERS paths. (#5325)

Azure · Feb 3, 2023 · 4149e70 · 4149e70
1 parent 4e4970c
commit 4149e70
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/tools/code-owners-parser/CodeOwnersParser/MatchedCodeownersEntry.cs b/tools/code-owners-parser/CodeOwnersParser/MatchedCodeownersEntry.cs
@@ -163,7 +163,8 @@ private static bool ContainsUnsupportedSequences(string codeownersPath)
             {
                 Console.Error.WriteLine(
                     $"CODEOWNERS path \"{codeownersPath}\" ends with " +
-                    "unsupported sequence of \"/**\". Replace it with \"/\".");
+                    "unsupported sequence of \"/**\". Replace it with \"/\". " +
+                    "Until then this path will never match.");
                 return true;
             }
 
@@ -173,7 +174,8 @@ private static bool ContainsUnsupportedSequences(string codeownersPath)
             {
                 Console.Error.WriteLine(
                     $"CODEOWNERS path \"{codeownersPath}\" ends with " +
-                    "unsupported sequence of \"/**/\". Replace it with \"/**\".");
+                    "unsupported sequence of \"/**/\". Replace it with \"/\". " +
+                    "Until then this path will never match.");
                 return true;
             }
 
@@ -184,8 +186,9 @@ private static bool ContainsUnsupportedSequences(string codeownersPath)
             {
                 Console.Error.WriteLine(
                     $"CODEOWNERS path \"{codeownersPath}\" contains " +
-                    "unsupported sequence of \"**\" that is not \"/**/\". Replace it with \"*\". " +
-                    "Until then this path will never match.");
+                    "unsupported sequence of \"**\". Double star can be used only within slashes \"/**/\" " +
+                    "or as a top-level catch all path of \"/**\". "  +
+                    "Currently this path will never match.");
                 return true;
             }