Fixes #2080: deprecate the top-level activeParameter property or update the documentation to clearly explain why there are two of them (#2211)

This commit is contained in:
Dirk Bäumer 2025-12-11 12:13:08 +01:00 committed by GitHub
parent 60f9a8ff39
commit 9e35a8d243
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 6 deletions

View file

@ -221,9 +221,9 @@ export interface SignatureHelp {
* lies outside the range of `signatures[activeSignature].parameters`
* defaults to 0 if the active signature has parameters. If
* the active signature has no parameters it is ignored.
* In future version of the protocol this property might become
* mandatory to better express the active parameter if the
* active signature does have any.
*
* Since version 3.16.0 the `SignatureInformation` itself provides a
* `activeParameter` property and it should be used instead of this one.
*/
activeParameter?: uinteger;
}

View file

@ -240,9 +240,8 @@ export interface SignatureHelp {
*
* If the active signature has no parameters, it is ignored.
*
* In future versions of the protocol this property might become
* mandatory (but still nullable) to better express the active parameter if
* the active signature does have any.
* Since version 3.16.0 the `SignatureInformation` itself provides a
* `activeParameter` property and it should be used instead of this one.
*/
activeParameter?: uinteger | null;
}