Skip to content

Commit 62c3974

Browse files
committed
Check for undefined, null or zero repositories
`undefined`, `null` and 0 will evaluate to `false` so if we only want to display the repository count when these values are not present we can check for a truthy value: ``` query.repositoryCount ? `(${pluralize(...)})` : ''; ``` instead of checking explicitly: ``` query.repositoryCount !== undefined && query.repositoryCount !== null && query.repositoryCount != 0 ? `(${pluralize(...)})` : ''; ```
1 parent ab1c2e0 commit 62c3974

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ export async function exportResultsToGist(
110110
const buildGistDescription = (query: RemoteQuery, analysesResults: AnalysisResults[]) => {
111111
const resultCount = sumAnalysesResults(analysesResults);
112112
const resultLabel = pluralize(resultCount, 'result', 'results');
113-
const repositoryLabel = query.repositoryCount !== undefined ? `(${pluralize(query.repositoryCount, 'repository', 'repositories')})` : '';
113+
const repositoryLabel = query.repositoryCount ? `(${pluralize(query.repositoryCount, 'repository', 'repositories')})` : '';
114114
return `${query.queryName} (${query.language}) ${resultLabel} ${repositoryLabel}`;
115115
};
116116

0 commit comments

Comments
 (0)