Skip to content

Commit cf50a71

Browse files
committed
Remove GitHub database download feature flag
This removes the `codeQL.githubDatabase.enable` setting and always enables the GitHub database download feature.
1 parent 598f2eb commit cf50a71

2 files changed

Lines changed: 0 additions & 11 deletions

File tree

extensions/ql-vscode/src/config.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -762,8 +762,6 @@ export class ModelConfigListener extends ConfigListener implements ModelConfig {
762762

763763
const GITHUB_DATABASE_SETTING = new Setting("githubDatabase", ROOT_SETTING);
764764

765-
// Feature flag for the GitHub database downnload.
766-
const GITHUB_DATABASE_ENABLE = new Setting("enable", GITHUB_DATABASE_SETTING);
767765
const GITHUB_DATABASE_DOWNLOAD = new Setting(
768766
"download",
769767
GITHUB_DATABASE_SETTING,
@@ -778,7 +776,6 @@ const GitHubDatabaseUpdateValues = ["ask", "never"] as const;
778776
type GitHubDatabaseUpdate = (typeof GitHubDatabaseUpdateValues)[number];
779777

780778
export interface GitHubDatabaseConfig {
781-
enable: boolean;
782779
download: GitHubDatabaseDownload;
783780
update: GitHubDatabaseUpdate;
784781
setDownload(
@@ -802,10 +799,6 @@ export class GitHubDatabaseConfigListener
802799
);
803800
}
804801

805-
public get enable() {
806-
return !!GITHUB_DATABASE_ENABLE.getValue<boolean>();
807-
}
808-
809802
public get download(): GitHubDatabaseDownload {
810803
const value = GITHUB_DATABASE_DOWNLOAD.getValue<GitHubDatabaseDownload>();
811804
return GitHubDatabaseDownloadValues.includes(value) ? value : "ask";

extensions/ql-vscode/src/databases/github-databases/github-databases-module.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,10 +60,6 @@ export class GitHubDatabasesModule extends DisposableObject {
6060
}
6161

6262
private async initialize(): Promise<void> {
63-
if (!this.config.enable) {
64-
return;
65-
}
66-
6763
// Start the check and downloading the database asynchronously. We don't want to block on this
6864
// in extension activation since this makes network requests and waits for user input.
6965
void this.promptGitHubRepositoryDownload().catch((e: unknown) => {

0 commit comments

Comments
 (0)