Skip to content

Commit c384a63

Browse files
Handle missing repo count gracefully
Let's handle this case gracefully and skip displaying the number of repositories when they're not available. Similarly let's add a check to see if we should pluralize the `repository` noun or not. Co-authored-by: Shati Patel <shati-patel@github.com>
1 parent b079690 commit c384a63

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

extensions/ql-vscode/src/remote-queries/export-results.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,9 @@ async function exportResultsToGist(
106106
*/
107107
const buildGistDescription = (query: RemoteQuery, analysesResults: AnalysisResults[]) => {
108108
const resultCount = sumAnalysesResults(analysesResults);
109-
return `${query.queryName} (${query.language}) ${resultCount} results (${query.numRepositoriesQueried} repositories)`;
109+
const repositoryLabel = `${query.numRepositoriesQueried} ${query.numRepositoriesQueried === 1 ? 'repository' : 'repositories'}`;
110+
const repositoryCount = query.numRepositoriesQueried ? repositoryLabel : '';
111+
return `${query.queryName} (${query.language}) ${resultCount} results (${repositoryCount})`;
110112
};
111113

112114
/**

0 commit comments

Comments
 (0)