Skip to content

Commit 155b83b

Browse files
Introduce remoteResultsItemWithoutLogs context value
1 parent 56111b3 commit 155b83b

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

extensions/ql-vscode/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -825,12 +825,12 @@
825825
{
826826
"command": "codeQLQueryHistory.removeHistoryItem",
827827
"group": "7_queryHistory@0",
828-
"when": "viewItem == interpretedResultsItem || viewItem == rawResultsItem || viewItem == remoteResultsItem || viewItem == cancelledResultsItem || viewItem == cancelledRemoteResultsItem"
828+
"when": "viewItem == interpretedResultsItem || viewItem == rawResultsItem || viewItem == remoteResultsItem || viewItem == cancelledRemoteResultsItemWithoutLogs || viewItem == cancelledResultsItem || viewItem == cancelledRemoteResultsItem"
829829
},
830830
{
831831
"command": "codeQLQueryHistory.removeHistoryItem",
832832
"group": "inline",
833-
"when": "viewItem == interpretedResultsItem || viewItem == rawResultsItem || viewItem == remoteResultsItem || viewItem == cancelledResultsItem || viewItem == cancelledRemoteResultsItem"
833+
"when": "viewItem == interpretedResultsItem || viewItem == rawResultsItem || viewItem == remoteResultsItem || viewItem == cancelledRemoteResultsItemWithoutLogs || viewItem == cancelledResultsItem || viewItem == cancelledRemoteResultsItem"
834834
},
835835
{
836836
"command": "codeQLQueryHistory.renameItem",

extensions/ql-vscode/src/query-history/history-tree-data-provider.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -111,9 +111,14 @@ export class HistoryTreeDataProvider
111111
return "remoteResultsItem";
112112
}
113113
case QueryStatus.Failed:
114-
return element.t === "local"
115-
? "cancelledResultsItem"
116-
: "cancelledRemoteResultsItem";
114+
if (element.t === "local") {
115+
return "cancelledResultsItem";
116+
} else if (element.variantAnalysis.actionsWorkflowRunId === undefined) {
117+
return "cancelledRemoteResultsItemWithoutLogs";
118+
} else {
119+
return "cancelledRemoteResultsItem";
120+
}
121+
117122
default:
118123
assertNever(element.status);
119124
}

0 commit comments

Comments
 (0)