Use correct branch name in config.

When storing the related PR number in the branch config, use the correct
branch name - this ensures that they key will be deleted along with the
branch. For some reason I thought this didn't work before, but it seems
to now?!

Author: grokys

src/GitHub.App/Services/PullRequestService.cs

@@ -160,7 +160,7 @@ public IObservable<Unit> SwitchToBranch(ILocalRepositoryModel repository, int nu
         async Task DoFetchAndCheckout(ILocalRepositoryModel repository, int pullRequestNumber, string localBranchName)
         {
             var repo = gitService.GetRepository(repository.LocalPath);
-            var configKey = $"branch.{BranchNameToConfigKey(localBranchName)}.ghfvs-pr";
+            var configKey = $"branch.{localBranchName}.ghfvs-pr";
             await gitClient.Fetch(repo, "origin", new[] { $"refs/pull/{pullRequestNumber}/head:{localBranchName}" });
             await gitClient.Checkout(repo, localBranchName);
             await gitClient.SetConfig(repo, configKey, pullRequestNumber.ToString());
@@ -172,7 +172,7 @@ IEnumerable<string> GetLocalBranchesInternal(IRepository repository, int number)
             return repository.Config
                 .Select(x => new { Branch = BranchCapture.Match(x.Key).Groups["branch"].Value, Value = x.Value })
                 .Where(x => !string.IsNullOrWhiteSpace(x.Branch) && x.Value == pr)
-                .Select(x => ConfigKeyToBranchName(x.Branch));
+                .Select(x => x.Branch);
         }
 
         async Task<IPullRequestModel> PushAndCreatePR(IRepositoryHost host,
@@ -224,9 +224,5 @@ static string GetSafeBranchName(string name)
                 before = after;
             }
         }
-
-        static string BranchNameToConfigKey(string name) => name.Replace("/", ".");
-
-        static string ConfigKeyToBranchName(string name) => name.Replace(".", "/");
     }
 }