Skip to content

Commit 15f1e78

Browse files
committed
treat signature help as a stable feature
1 parent afa4207 commit 15f1e78

File tree

4 files changed

+11
-7
lines changed

4 files changed

+11
-7
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@
1616

1717
- Enable completion for `Js.Exn.Error(error)` when pattern matching on `exn`. This is to make the `Js.Exn.Error` API more discoverable. https://github.com/rescript-lang/rescript-vscode/pull/728
1818

19+
#### :nail_care: Polish
20+
21+
- Signature Help is now considered stable, and enabled for all users. Can still be turned off in settings.
22+
1923
## 1.12.0
2024

2125
#### :rocket: New Feature

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ You'll find all ReScript specific settings under the scope `rescript.settings`.
104104
| ReScript Platform Path | The extension will look for the existence of a `node_modules/rescript` directory and use the subdirectory corresponding to the current platform as the `platformPath`. If it does not find it at the project root (which is where the nearest `bsconfig.json` resides), it goes up folders in the filesystem recursively until it either finds it (often the case in monorepos) or hits the top level. To override this lookup process, the path can be configured explicitly using the setting `rescript.settings.platformPath` |
105105
| Inlay Hints (experimental) | This allows an editor to place annotations inline with text to display type hints. Enable using `rescript.settings.inlayHints.enable: true` |
106106
| Code Lens (experimental) | This tells the editor to add code lenses to function definitions, showing its full type above the definition. Enable using `rescript.settings.codeLens: true` |
107-
| Signature Help (experimental) | This tells the editor to show signature help when you're writing function calls. Enable using `rescript.settings.signatureHelp.enable: true` |
107+
| Signature Help | This tells the editor to show signature help when you're writing function calls. Enable using `rescript.settings.signatureHelp.enable: true` |
108108
| Autostarting the Code Analyzer | The Code Analyzer needs to be started manually by default. However, you can configure the extension to start the Code Analyzer automatically via the setting `rescript.settings.autoRunCodeAnalysis`. |
109109

110110
**Default settings:**

package.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -158,10 +158,10 @@
158158
"default": false,
159159
"description": "Enable (experimental) code lens for function definitions."
160160
},
161-
"rescript.settings.signatureHelp.enable": {
161+
"rescript.settings.signatureHelp.enabled": {
162162
"type": "boolean",
163-
"default": false,
164-
"description": "Enable (experimental) signature help for function calls."
163+
"default": true,
164+
"description": "Enable signature help for function calls."
165165
},
166166
"rescript.settings.binaryPath": {
167167
"type": ["string", "null"],

server/src/server.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ interface extensionConfiguration {
3737
binaryPath: string | null;
3838
platformPath: string | null;
3939
signatureHelp: {
40-
enable: boolean;
40+
enabled: boolean;
4141
};
4242
}
4343

@@ -63,7 +63,7 @@ let extensionConfiguration: extensionConfiguration = {
6363
binaryPath: null,
6464
platformPath: null,
6565
signatureHelp: {
66-
enable: false,
66+
enabled: true,
6767
},
6868
};
6969
// Below here is some state that's not important exactly how long it lives.
@@ -1145,7 +1145,7 @@ function onMessage(msg: p.Message) {
11451145
workDoneProgress: false,
11461146
}
11471147
: undefined,
1148-
signatureHelpProvider: extensionConfiguration.signatureHelp?.enable
1148+
signatureHelpProvider: extensionConfiguration.signatureHelp?.enabled
11491149
? {
11501150
triggerCharacters: ["("],
11511151
retriggerCharacters: ["=", ","],

0 commit comments

Comments
 (0)