Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit acc9480

Browse files
authored
Merge branch 'master' into fixes/1644-GitServiceHelper-avoid-MEF
2 parents d7e69fd + c3d8441 commit acc9480

File tree

5 files changed

+20
-8
lines changed

5 files changed

+20
-8
lines changed

src/GitHub.Exports/Models/UsageModel.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,8 @@ public class MeasuresModel
7070
public int NumberOfShowCurrentPullRequest { get; set; }
7171
public int NumberOfStatusBarOpenPullRequestList { get; set; }
7272
public int NumberOfTeamExplorerHomeOpenPullRequestList { get; set; }
73+
public int NumberOfStartPageClones { get; set; }
74+
public int NumberOfGitHubConnectSectionClones { get; set; }
7375
public int NumberOfShowRepoForkDialogClicks { get; set; }
7476
public int NumberOfReposForked { get; set; }
7577
public int NumberOfOriginsUpdatedWhenForkingRepo { get; set; }

src/GitHub.StartPage/StartPagePackage.cs

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
using Microsoft.TeamFoundation.Controls;
1313
using Microsoft.VisualStudio.Shell;
1414
using Microsoft.VisualStudio.Shell.CodeContainerManagement;
15+
using Microsoft.VisualStudio.Threading;
1516
using Serilog;
1617
using CodeContainer = Microsoft.VisualStudio.Shell.CodeContainerManagement.CodeContainer;
1718
using ICodeContainerProvider = Microsoft.VisualStudio.Shell.CodeContainerManagement.ICodeContainerProvider;
@@ -119,8 +120,9 @@ async Task<CloneDialogResult> ShowCloneDialog(
119120
{
120121
var dialogService = gitHubServiceProvider.GetService<IDialogService>();
121122
var cloneService = gitHubServiceProvider.GetService<IRepositoryCloneService>();
123+
var usageTracker = gitHubServiceProvider.GetService<IUsageTracker>();
122124
CloneDialogResult result = null;
123-
125+
124126
if (repository == null)
125127
{
126128
result = await dialogService.ShowCloneDialog(null);
@@ -134,7 +136,7 @@ async Task<CloneDialogResult> ShowCloneDialog(
134136
result = new CloneDialogResult(basePath, repository);
135137
}
136138
}
137-
139+
138140
if (result != null)
139141
{
140142
try
@@ -144,6 +146,8 @@ await cloneService.CloneRepository(
144146
result.Repository.Name,
145147
result.BasePath,
146148
progress);
149+
150+
usageTracker.IncrementCounter(x => x.NumberOfStartPageClones).Forget();
147151
}
148152
catch
149153
{

src/GitHub.TeamFoundation.14/Connect/GitHubConnectSection.cs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ public class GitHubConnectSection : TeamExplorerSectionBase, IGitHubConnectSecti
3333
readonly IVSServices vsServices;
3434
readonly int sectionIndex;
3535
readonly ILocalRepositories localRepositories;
36+
readonly IUsageTracker usageTracker;
3637

3738
bool isCloning;
3839
bool isCreating;
@@ -99,6 +100,7 @@ public GitHubConnectSection(IGitHubServiceProvider serviceProvider,
99100
IPackageSettings packageSettings,
100101
IVSServices vsServices,
101102
ILocalRepositories localRepositories,
103+
IUsageTracker usageTracker,
102104
int index)
103105
: base(serviceProvider, apiFactory, holder, manager)
104106
{
@@ -108,6 +110,7 @@ public GitHubConnectSection(IGitHubServiceProvider serviceProvider,
108110
Guard.ArgumentNotNull(packageSettings, nameof(packageSettings));
109111
Guard.ArgumentNotNull(vsServices, nameof(vsServices));
110112
Guard.ArgumentNotNull(localRepositories, nameof(localRepositories));
113+
Guard.ArgumentNotNull(usageTracker, nameof(usageTracker));
111114

112115
Title = "GitHub";
113116
IsEnabled = true;
@@ -118,6 +121,7 @@ public GitHubConnectSection(IGitHubServiceProvider serviceProvider,
118121
this.packageSettings = packageSettings;
119122
this.vsServices = vsServices;
120123
this.localRepositories = localRepositories;
124+
this.usageTracker = usageTracker;
121125

122126
Clone = CreateAsyncCommandHack(DoClone);
123127

@@ -141,6 +145,8 @@ await cloneService.CloneRepository(
141145
result.Repository.CloneUrl,
142146
result.Repository.Name,
143147
result.BasePath);
148+
149+
usageTracker.IncrementCounter(x => x.NumberOfGitHubConnectSectionClones).Forget();
144150
}
145151
catch (Exception e)
146152
{

src/GitHub.TeamFoundation.14/Connect/GitHubConnectSection0.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
using GitHub.Api;
2-
using GitHub.Models;
32
using GitHub.Services;
43
using GitHub.Settings;
54
using Microsoft.TeamFoundation.Controls;
@@ -20,8 +19,9 @@ public GitHubConnectSection0(IGitHubServiceProvider serviceProvider,
2019
IConnectionManager manager,
2120
IPackageSettings settings,
2221
IVSServices vsServices,
23-
ILocalRepositories localRepositories)
24-
: base(serviceProvider, apiFactory, holder, manager, settings, vsServices, localRepositories, 0)
22+
ILocalRepositories localRepositories,
23+
IUsageTracker usageTracker)
24+
: base(serviceProvider, apiFactory, holder, manager, settings, vsServices, localRepositories, usageTracker, 0)
2525
{
2626
}
2727
}

src/GitHub.TeamFoundation.14/Connect/GitHubConnectSection1.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
using GitHub.Api;
2-
using GitHub.Models;
32
using GitHub.Services;
43
using GitHub.Settings;
54
using Microsoft.TeamFoundation.Controls;
@@ -20,8 +19,9 @@ public GitHubConnectSection1(IGitHubServiceProvider serviceProvider,
2019
IConnectionManager manager,
2120
IPackageSettings settings,
2221
IVSServices vsServices,
23-
ILocalRepositories localRepositories)
24-
: base(serviceProvider, apiFactory, holder, manager, settings, vsServices, localRepositories, 1)
22+
ILocalRepositories localRepositories,
23+
IUsageTracker usageTracker)
24+
: base(serviceProvider, apiFactory, holder, manager, settings, vsServices, localRepositories, usageTracker, 1)
2525
{
2626
}
2727
}

0 commit comments

Comments
 (0)