vscode: rename alwaysDownloadServer -> askBeforeDownload

The new name seems much simpler and it doesn't limit
this config value only to downloading the server binary.
Thus we wouldn't need to create another config
properties to handle other downloads whatsoever.
Anyway, I believe (heuristically) that most of the users
would want to set "askBeforeDownload": false once
and never bother clicking on the notification again
(because otherwise there is no big point in installing rust-analyzer if it cannot install the server)

2 files changed, 4 insertions, 4 deletions

diff --git a/editors/code/package.json b/editors/code/package.json
index 7bcd45e15..430a61c64 100644
--- a/editors/code/package.json
+++ b/editors/code/package.json
@@ -219,10 +219,10 @@
                         }
                     }
                 },
-                "rust-analyzer.alwaysDownloadServer": {
+                "rust-analyzer.askBeforeDownload": {
                     "type": "boolean",
-                    "default": false,
-                    "description": "Whether to ask before downloading the language server binary"
+                    "default": true,
+                    "description": "Whether to ask for permission before downloading any files from the Internet"
                 },
                 "rust-analyzer.serverPath": {
                     "type": [
diff --git a/editors/code/src/config.ts b/editors/code/src/config.ts
index c4acb632d..cb9af86c0 100644
--- a/editors/code/src/config.ts
+++ b/editors/code/src/config.ts
@@ -134,7 +134,7 @@ export class Config {
             file: prebuiltBinaryName,
             storage: this.ctx.globalState,
             tag: Config.extensionVersion,
-            askBeforeDownload: !(this.cfg.get("alwaysDownloadServer") as boolean),
+            askBeforeDownload: !(this.cfg.get("askBeforeDownload") as boolean),
             repo: {
                 name: "rust-analyzer",
                 owner: "rust-analyzer",