Removing unused metrics

Author: StanleyGoldman

src/GitHub.App/Services/RepositoryForkService.cs

@@ -41,7 +41,7 @@ public IObservable<Repository> ForkRepository(IApiClient apiClient, IRepositoryM
             log.Verbose("ForkRepository Source:{SourceOwner}/{SourceName} To:{DestinationOwner}", sourceRepository.Owner, sourceRepository.Name, repositoryFork.Organization ?? "[Current User]");
             log.Verbose("ForkRepository updateOrigin:{UpdateOrigin} addUpstream:{AddUpstream} trackMasterUpstream:{TrackMasterUpstream}", updateOrigin, addUpstream, trackMasterUpstream);
 
-            RecordForkRepositoryUsage(updateOrigin, addUpstream, trackMasterUpstream).Forget();
+            usageTracker.IncrementCounter(model => model.NumberOfReposForked).Forget();
 
             return Observable.Defer(() => apiClient.ForkRepository(sourceRepository.Owner, sourceRepository.Name, repositoryFork)
                     .ObserveOn(RxApp.MainThreadScheduler)
@@ -63,26 +63,6 @@ public IObservable<Repository> ForkRepository(IApiClient apiClient, IRepositoryM
                 });
         }
 
-        private async Task RecordForkRepositoryUsage(bool updateOrigin, bool addUpstream, bool trackMasterUpstream)
-        {
-            await usageTracker.IncrementCounter(model => model.NumberOfReposForked);
-
-            if (updateOrigin)
-            {
-                await usageTracker.IncrementCounter(model => model.NumberOfOriginsUpdatedWhenForkingRepo);
-            }
-
-            if (addUpstream)
-            {
-                await usageTracker.IncrementCounter(model => model.NumberOfUpstreamsAddedWhenForkingRepo);
-            }
-
-            if (trackMasterUpstream)
-            {
-                await usageTracker.IncrementCounter(model => model.NumberOfTrackMasterUpstreamWhenForkingRepo);
-            }
-        }
-
         public IObservable<object> SwitchRemotes(IRepositoryModel destinationRepository, bool updateOrigin, bool addUpstream, bool trackMasterUpstream)
         {
             return Observable.Defer(() => Observable.Return(new object())

src/GitHub.Exports/Models/UsageModel.cs

@@ -72,9 +72,6 @@ public class MeasuresModel
             public int NumberOfTeamExplorerHomeOpenPullRequestList { get; set; }
             public int NumberOfShowRepoForkDialogClicks { get; set; }
             public int NumberOfReposForked { get; set; }
-            public int NumberOfOriginsUpdatedWhenForkingRepo { get; set; }
-            public int NumberOfUpstreamsAddedWhenForkingRepo { get; set; }
-            public int NumberOfTrackMasterUpstreamWhenForkingRepo { get; set; }
         }
     }
 }