Skip to content

Commit

Permalink
Remove duplicate GitHubService and netfx nuget dependency
Browse files Browse the repository at this point in the history
  • Loading branch information
hallipr committed Jul 28, 2022
1 parent a6659df commit a957cbb
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 76 deletions.
1 change: 0 additions & 1 deletion tools/identity-resolution/identity-resolution.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
<ItemGroup>
<PackageReference Include="Microsoft.Extensions.Logging" Version="2.2.0" />
<PackageReference Include="Microsoft.TeamFoundationServer.Client" Version="16.170.0" />
<PackageReference Include="Microsoft.TeamFoundationServer.ExtendedClient" Version="16.170.0" />
<PackageReference Include="Microsoft.VisualStudio.Services.Notifications.WebApi" Version="16.170.0" />
<PackageReference Include="YamlDotNet" Version="6.1.1" />
<PackageReference Include="Azure.Identity" Version="1.5.0" />
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
using System.Threading.Tasks;

using Azure.Sdk.Tools.CodeOwnersParser;
using Azure.Sdk.Tools.NotificationConfiguration;
using Azure.Sdk.Tools.NotificationConfiguration.Helpers;
using Azure.Sdk.Tools.NotificationConfiguration.Services;
using Azure.Sdk.Tools.PipelineOwnersExtractor.Configuration;
Expand Down Expand Up @@ -136,7 +137,7 @@ private async Task<Dictionary<string, List<CodeOwnerEntry>>> GetCodeOwnerEntries
.Select(SanitizeRepositoryUrl)
.Select(async url => (
RepositoryUrl: url,
CodeOwners: await this.gitHubService.GetCodeOwnersFile(new Uri(url))
CodeOwners: await this.gitHubService.GetCodeownersFile(new Uri(url))
));

var taskResults = await Task.WhenAll(tasks);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
using System.Threading.Tasks;
using Azure.Core;
using Azure.Identity;
using Azure.Sdk.Tools.NotificationConfiguration;
using Azure.Sdk.Tools.NotificationConfiguration.Helpers;
using Azure.Sdk.Tools.NotificationConfiguration.Services;
using Azure.Sdk.Tools.PipelineOwnersExtractor.Configuration;
Expand Down

0 comments on commit a957cbb

Please sign in to comment.