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

Commit 8a7454f

Browse files
Merge branch 'features/check-suite-annotations' into features/check-suite-annotations-inline
2 parents 80d5b9a + 0af2495 commit 8a7454f

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

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

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ public class PullRequestAnnotationsViewModel : PanePageViewModelBase, IPullReque
2121
readonly IPullRequestSessionManager sessionManager;
2222
readonly IPullRequestEditorService pullRequestEditorService;
2323

24-
IPullRequestSession session;
2524
string title;
2625
string checkSuiteName;
2726
string checkRunName;
@@ -60,8 +59,8 @@ public async Task InitializeAsync(ILocalRepositoryModel localRepository, IConnec
6059
RemoteRepositoryOwner = owner;
6160
PullRequestNumber = pullRequestNumber;
6261
CheckRunId = checkRunId;
63-
session = await sessionManager.GetSession(owner, repo, pullRequestNumber);
64-
await Load(session.PullRequest);
62+
var session = await sessionManager.GetSession(owner, repo, pullRequestNumber);
63+
Load(session.PullRequest);
6564
}
6665
finally
6766
{
@@ -112,19 +111,18 @@ public IReadOnlyList<IPullRequestAnnotationItemViewModel> Annotations
112111
private set { this.RaiseAndSetIfChanged(ref annotations, value); }
113112
}
114113

115-
async Task Load(PullRequestDetailModel pullRequest)
114+
void Load(PullRequestDetailModel pullRequest)
116115
{
117116
IsBusy = true;
118117

119118
try
120119
{
121-
await Task.Delay(0);
122120
PullRequestTitle = pullRequest.Title;
123121

124122
var checkSuiteRun = pullRequest
125123
.CheckSuites.SelectMany(checkSuite => checkSuite.CheckRuns
126124
.Select(checkRun => new{checkSuite, checkRun}))
127-
.First(arg => arg.checkRun.DatabaseId == CheckRunId);
125+
.First(arg => arg.checkRun.Name == CheckRunName);
128126

129127
CheckSuiteName = checkSuiteRun.checkSuite.ApplicationName;
130128
CheckRunName = checkSuiteRun.checkRun.Name;

0 commit comments

Comments
 (0)