Replace the glob-based CODEOWNERS parser with regex-based (#5063)

#5063
Azure · Jan 9, 2023 · 4b7f14b · 4b7f14b
1 parent 6b3ee03
commit 4b7f14b
Show file tree

Hide file tree

Showing 7 changed files with 308 additions and 245 deletions.
diff --git a/tools/code-owners-parser/Azure.Sdk.Tools.CodeOwnersParser.Tests/CodeOwnersFileTests.cs b/tools/code-owners-parser/Azure.Sdk.Tools.CodeOwnersParser.Tests/CodeOwnersFileTests.cs
diff --git a/tools/code-owners-parser/Azure.Sdk.Tools.RetrieveCodeOwners/Program.cs b/tools/code-owners-parser/Azure.Sdk.Tools.RetrieveCodeOwners/Program.cs
@@ -23,7 +23,7 @@ public static int Main(
             bool filterOutNonUserAliases = false
             )
         {
-            var target = targetDirectory.ToLower().Trim();
+            var target = targetDirectory.Trim();
             try {
                 var codeOwnerEntry = CodeOwnersFile.ParseAndFindOwnersForClosestMatch(codeOwnerFilePath, target);
                 if (filterOutNonUserAliases)

diff --git a/tools/code-owners-parser/CodeOwnersParser/Azure.Sdk.Tools.CodeOwnersParser.csproj b/tools/code-owners-parser/CodeOwnersParser/Azure.Sdk.Tools.CodeOwnersParser.csproj
@@ -7,7 +7,7 @@
 
   <ItemGroup>
     <PackageReference Include="CommandLine.Net" Version="2.3.0" />
-    <PackageReference Include="Microsoft.Extensions.FileSystemGlobbing" Version="7.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="7.0.0" />
   </ItemGroup>
 
 </Project>
diff --git a/tools/code-owners-parser/CodeOwnersParser/MatchedCodeOwnerEntry.cs b/tools/code-owners-parser/CodeOwnersParser/MatchedCodeOwnerEntry.cs
diff --git a/tools/identity-resolution/identity-resolution.csproj b/tools/identity-resolution/identity-resolution.csproj
@@ -6,7 +6,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.Extensions.Logging" Version="2.2.0" />
+    <PackageReference Include="Microsoft.Extensions.Logging" Version="7.0.0" />
     <PackageReference Include="Microsoft.TeamFoundationServer.Client" Version="16.170.0" />
     <PackageReference Include="Microsoft.VisualStudio.Services.Notifications.WebApi" Version="16.170.0" />
     <PackageReference Include="YamlDotNet" Version="6.1.1" />

diff --git a/...ation-configuration/notification-creator/Azure.Sdk.Tools.NotificationConfiguration.csproj b/...ation-configuration/notification-creator/Azure.Sdk.Tools.NotificationConfiguration.csproj
@@ -9,7 +9,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="5.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="7.0.0" />
     <PackageReference Include="System.CommandLine.DragonFruit" Version="0.3.0-alpha.19317.1" />
   </ItemGroup>
 

diff --git a/tools/notification-configuration/notification-creator/Program.cs b/tools/notification-configuration/notification-creator/Program.cs
@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Threading.Tasks;
 using Microsoft.Extensions.Logging;
 using Microsoft.VisualStudio.Services.Common;
@@ -39,12 +39,10 @@ static async Task Main(
             var devOpsCreds = new VssBasicCredential("nobody", devOpsToken);
             var devOpsConnection = new VssConnection(new Uri($"https://dev.azure.com/{organization}/"), devOpsCreds);
 
-#pragma warning disable CS0618 // Type or member is obsolete
             var loggerFactory = LoggerFactory.Create(builder =>
             {
-                builder.AddConsole(config => { config.IncludeScopes = true; });
+                builder.AddSimpleConsole(config => { config.IncludeScopes = true; });
             });
-#pragma warning restore CS0618 // Type or member is obsolete
             var devOpsServiceLogger = loggerFactory.CreateLogger<AzureDevOpsService>();
             var notificationConfiguratorLogger = loggerFactory.CreateLogger<NotificationConfigurator>();