Skip to content

Commit a6b3d38

Browse files
authored
Re-use setRepoResults instead of having a setReposResult message (#3506)
1 parent 1739af4 commit a6b3d38

3 files changed

Lines changed: 4 additions & 14 deletions

File tree

extensions/ql-vscode/src/common/interface-types.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -732,11 +732,6 @@ interface SetModelAlertsViewStateMessage {
732732
viewState: ModelAlertsViewState;
733733
}
734734

735-
interface SetReposResultsMessage {
736-
t: "setReposResults";
737-
reposResults: VariantAnalysisScannedRepositoryResult[];
738-
}
739-
740735
interface OpenModelPackMessage {
741736
t: "openModelPack";
742737
path: string;
@@ -754,8 +749,7 @@ interface StopEvaluationRunMessage {
754749
export type ToModelAlertsMessage =
755750
| SetModelAlertsViewStateMessage
756751
| SetVariantAnalysisMessage
757-
| SetRepoResultsMessage
758-
| SetReposResultsMessage;
752+
| SetRepoResultsMessage;
759753

760754
export type FromModelAlertsMessage =
761755
| CommonFromViewMessages

extensions/ql-vscode/src/model-editor/model-alerts/model-alerts-view.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -143,15 +143,15 @@ export class ModelAlertsView extends AbstractWebview<
143143
}
144144

145145
public async updateReposResults(
146-
reposResults: VariantAnalysisScannedRepositoryResult[],
146+
repoResults: VariantAnalysisScannedRepositoryResult[],
147147
): Promise<void> {
148148
if (!this.isShowingPanel) {
149149
return;
150150
}
151151

152152
await this.postMessage({
153-
t: "setReposResults",
154-
reposResults,
153+
t: "setRepoResults",
154+
repoResults,
155155
});
156156
}
157157

extensions/ql-vscode/src/view/model-alerts/ModelAlerts.tsx

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,10 +66,6 @@ export function ModelAlerts({
6666
setVariantAnalysis(msg.variantAnalysis);
6767
break;
6868
}
69-
case "setReposResults": {
70-
setRepoResults(msg.reposResults);
71-
break;
72-
}
7369
case "setRepoResults": {
7470
setRepoResults((oldRepoResults) => {
7571
const newRepoIds = msg.repoResults.map((r) => r.repositoryId);

0 commit comments

Comments
 (0)