Skip to content

Commit 798714c

Browse files
committed
Rename config option to say "default"
1 parent 1970af9 commit 798714c

4 files changed

Lines changed: 12 additions & 12 deletions

File tree

extensions/ql-vscode/CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
## [UNRELEASED]
44

5-
- Add settings `codeQL.variantAnalysis.filterResults` and `codeQL.variantAnalysis.sortResults` for configuring how variant analysis results are filtered and sorted in the results view. The default is to show only repositories with results, and to sort by the number of results. [#2392](https://github.com/github/vscode-codeql/pull/2392)
5+
- Add settings `codeQL.variantAnalysis.defaultResultsFilter` and `codeQL.variantAnalysis.defaultResultsSort` for configuring how variant analysis results are filtered and sorted in the results view. The default is to show only repositories with results, and to sort by the number of results. [#2392](https://github.com/github/vscode-codeql/pull/2392)
66
- On the variant analysis results page, show the count of successful analyses instead of completed analyses, and indicate the reason why analyses were not successful. [#2349](https://github.com/github/vscode-codeql/pull/2349)
77
- Fix bug where the "CodeQL: Set Current Database" command didn't always select the database. [#2384](https://github.com/github/vscode-codeql/pull/2384)
88

extensions/ql-vscode/package.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@
329329
"patternErrorMessage": "Please enter a valid GitHub repository",
330330
"markdownDescription": "[For internal use only] The name of the GitHub repository in which the GitHub Actions workflow is run when using the \"Run Variant Analysis\" command. The repository should be of the form `<owner>/<repo>`)."
331331
},
332-
"codeQL.variantAnalysis.filterResults": {
332+
"codeQL.variantAnalysis.defaultResultsFilter": {
333333
"type": "string",
334334
"default": "withResults",
335335
"enum": [
@@ -340,9 +340,9 @@
340340
"Show all repositories in the results view.",
341341
"Show only repositories with results in the results view."
342342
],
343-
"description": "The filter to apply to the variant analysis results view."
343+
"description": "The default filter to apply to the variant analysis results view."
344344
},
345-
"codeQL.variantAnalysis.sortResults": {
345+
"codeQL.variantAnalysis.defaultResultsSort": {
346346
"type": "string",
347347
"default": "numberOfResults",
348348
"enum": [

extensions/ql-vscode/src/config.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -535,11 +535,11 @@ export class VariantAnalysisConfigListener
535535
}
536536

537537
const VARIANT_ANALYSIS_FILTER_RESULTS = new Setting(
538-
"filterResults",
538+
"defaultResultsFilter",
539539
VARIANT_ANALYSIS_SETTING,
540540
);
541541

542-
export function getVariantAnalysisFilterResults(): FilterKey {
542+
export function getVariantAnalysisDefaultResultsFilter(): FilterKey {
543543
const value = VARIANT_ANALYSIS_FILTER_RESULTS.getValue<string>();
544544
if (Object.values(FilterKey).includes(value as FilterKey)) {
545545
return value as FilterKey;
@@ -549,11 +549,11 @@ export function getVariantAnalysisFilterResults(): FilterKey {
549549
}
550550

551551
const VARIANT_ANALYSIS_SORT_RESULTS = new Setting(
552-
"sortResults",
552+
"defaultResultsSort",
553553
VARIANT_ANALYSIS_SETTING,
554554
);
555555

556-
export function getVariantAnalysisSortResults(): SortKey {
556+
export function getVariantAnalysisDefaultResultsSort(): SortKey {
557557
const value = VARIANT_ANALYSIS_SORT_RESULTS.getValue<string>();
558558
if (Object.values(SortKey).includes(value as SortKey)) {
559559
return value as SortKey;

extensions/ql-vscode/src/variant-analysis/variant-analysis-view.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ import { DataFlowPathsView } from "./data-flow-paths-view";
2828
import { DataFlowPaths } from "./shared/data-flow-paths";
2929
import { App } from "../common/app";
3030
import {
31-
getVariantAnalysisFilterResults,
32-
getVariantAnalysisSortResults,
31+
getVariantAnalysisDefaultResultsFilter,
32+
getVariantAnalysisDefaultResultsSort,
3333
} from "../config";
3434

3535
export class VariantAnalysisView
@@ -192,8 +192,8 @@ export class VariantAnalysisView
192192

193193
const filterSortState = {
194194
searchValue: "",
195-
filterKey: getVariantAnalysisFilterResults(),
196-
sortKey: getVariantAnalysisSortResults(),
195+
filterKey: getVariantAnalysisDefaultResultsFilter(),
196+
sortKey: getVariantAnalysisDefaultResultsSort(),
197197
};
198198

199199
await this.postMessage({

0 commit comments

Comments
 (0)