Merge branch 'master' into refactor/mvvm

 Conflicts:
	src/GitHub.App/ViewModels/GitHubPane/PullRequestDetailViewModel.cs

Author: grokys

src/GitHub.App/ViewModels/GitHubPane/PullRequestDetailViewModel.cs

@@ -24,14 +24,15 @@ namespace GitHub.ViewModels.GitHubPane
     /// </summary>
     [Export(typeof(IPullRequestDetailViewModel))]
     [PartCreationPolicy(CreationPolicy.NonShared)]
-    public class PullRequestDetailViewModel : PanePageViewModelBase, IPullRequestDetailViewModel
+    public sealed class PullRequestDetailViewModel : PanePageViewModelBase, IPullRequestDetailViewModel
     {
         static readonly ILogger log = LogManager.ForContext<PullRequestDetailViewModel>();
 
         readonly IModelServiceFactory modelServiceFactory;
         readonly IPullRequestService pullRequestsService;
         readonly IPullRequestSessionManager sessionManager;
         readonly IUsageTracker usageTracker;
+        readonly IVSGitExt vsGitExt;
         IModelService modelService;
         IPullRequestModel model;
         string sourceBranchDisplayName;
@@ -55,12 +56,14 @@ public class PullRequestDetailViewModel : PanePageViewModelBase, IPullRequestDet
         /// <param name="pullRequestsService">The pull requests service.</param>
         /// <param name="sessionManager">The pull request session manager.</param>
         /// <param name="usageTracker">The usage tracker.</param>
+        /// <param name="vsGitExt">The Visual Studio git service.</param>
         [ImportingConstructor]
         public PullRequestDetailViewModel(
             IPullRequestService pullRequestsService,
             IPullRequestSessionManager sessionManager,
             IModelServiceFactory modelServiceFactory,
-            IUsageTracker usageTracker)
+            IUsageTracker usageTracker,
+            IVSGitExt vsGitExt)
         {
             Guard.ArgumentNotNull(pullRequestsService, nameof(pullRequestsService));
             Guard.ArgumentNotNull(sessionManager, nameof(sessionManager));
@@ -71,6 +74,7 @@ public PullRequestDetailViewModel(
             this.sessionManager = sessionManager;
             this.modelServiceFactory = modelServiceFactory;
             this.usageTracker = usageTracker;
+            this.vsGitExt = vsGitExt;
 
             Checkout = ReactiveCommand.CreateAsyncObservable(
                 this.WhenAnyValue(x => x.CheckoutState)
@@ -98,7 +102,7 @@ public PullRequestDetailViewModel(
             DiffFile = ReactiveCommand.Create();
             DiffFileWithWorkingDirectory = ReactiveCommand.Create(this.WhenAnyValue(x => x.IsCheckedOut));
             OpenFileInWorkingDirectory = ReactiveCommand.Create(this.WhenAnyValue(x => x.IsCheckedOut));
-            ViewFile = ReactiveCommand.Create();
+            ViewFile = ReactiveCommand.Create();            
         }
 
         /// <summary>
@@ -314,6 +318,7 @@ public async Task InitializeAsync(
                 Number = number;
                 WebUrl = LocalRepository.CloneUrl.ToRepositoryUrl().Append("pull/" + number);
                 modelService = await modelServiceFactory.CreateAsync(connection);
+                vsGitExt.ActiveRepositoriesChanged += ActiveRepositoriesChanged;
                 await Refresh();
             }
             finally
@@ -477,6 +482,19 @@ public string GetLocalFilePath(IPullRequestFileNode file)
             return Path.Combine(LocalRepository.LocalPath, file.DirectoryPath, file.FileName);
         }
 
+        /// <inheritdoc/>
+        protected override void Dispose(bool disposing)
+        {
+            base.Dispose(disposing);
+
+            if (disposing)
+            {
+                vsGitExt.ActiveRepositoriesChanged -= ActiveRepositoriesChanged;
+            }
+        }
+
+        void ActiveRepositoriesChanged() => Refresh().Forget();
+
         void SubscribeOperationError(ReactiveCommand<Unit> command)
         {
             command.ThrownExceptions.Subscribe(x => OperationError = x.Message);