Skip to content

Fixed wrong SignatureInformation.activeParameter comment#155279

Merged
mjbvz merged 1 commit into
microsoft:mainfrom
jbicker:signatureInformation-activeParameter-comment
Jul 15, 2022
Merged

Fixed wrong SignatureInformation.activeParameter comment#155279
mjbvz merged 1 commit into
microsoft:mainfrom
jbicker:signatureInformation-activeParameter-comment

Conversation

@jbicker
Copy link
Copy Markdown
Contributor

@jbicker jbicker commented Jul 15, 2022

This PR fixes the comment of SignatureInformation.activeParameter.

Spotted this in the docs.

Current:
If provided, this is used in place of SignatureHelp.activeSignature.

Expected:
If provided, this is used in place of SignatureHelp.activeParameter.

@mjbvz mjbvz enabled auto-merge (squash) July 15, 2022 17:27
@mjbvz mjbvz added this to the July 2022 milestone Jul 15, 2022
@mjbvz mjbvz merged commit f3b174e into microsoft:main Jul 15, 2022
jrieken pushed a commit that referenced this pull request Jul 18, 2022
Fixed SignatureInformation.activeParameter comment
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants