Using string.Equals instead

StanleyGoldman · StanleyGoldman · commit 3586304b5f93 · 2018-07-25T17:36:23.000-04:00
diff --git a/src/GitHub.App/Services/PullRequestService.cs b/src/GitHub.App/Services/PullRequestService.cs
@@ -524,7 +524,7 @@ public IObservable<bool> EnsureLocalBranchesAreMarkedAsPullRequests(ILocalReposi
 
         public bool IsPullRequestFromRepository(ILocalRepositoryModel repository, PullRequestDetailModel pullRequest)
         {
-            return pullRequest.HeadRepositoryOwner.ToLowerInvariant() == repository.CloneUrl.Owner.ToLowerInvariant();
+            return string.Equals(repository?.CloneUrl.Owner, pullRequest.HeadRepositoryOwner, StringComparison.OrdinalIgnoreCase);
         }
 
         public IObservable<Unit> SwitchToBranch(ILocalRepositoryModel repository, PullRequestDetailModel pullRequest)

Original file line number	Diff line number	Diff line change
`@@ -524,7 +524,7 @@ public IObservable<bool> EnsureLocalBranchesAreMarkedAsPullRequests(ILocalReposi`
`524`	`524`
`525`	`525`	`public bool IsPullRequestFromRepository(ILocalRepositoryModel repository, PullRequestDetailModel pullRequest)`
`526`	`526`	`{`
`527`		`- return pullRequest.HeadRepositoryOwner.ToLowerInvariant() == repository.CloneUrl.Owner.ToLowerInvariant();`
	`527`	`+ return string.Equals(repository?.CloneUrl.Owner, pullRequest.HeadRepositoryOwner, StringComparison.OrdinalIgnoreCase);`
`528`	`528`	`}`
`529`	`529`
`530`	`530`	`public IObservable<Unit> SwitchToBranch(ILocalRepositoryModel repository, PullRequestDetailModel pullRequest)`