Skip to content

Fix inlay hint crash for jsdoc function type syntax #47684

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 1, 2022

Conversation

sandersn
Copy link
Member

@sandersn sandersn commented Feb 1, 2022

Parameters in JSDoc function types do not have names. The type does not reflect this. This PR fixes the crash; I'll see how much churn it causes to fix the type as well.

Fixes #47606

Parameters in JSDoc function types do not have names. The type does not
reflect this. This PR fixes the crash; I'll see how much churn it causes
to fix the type as well.

Fixes #47606
@typescript-bot typescript-bot added Author: Team For Milestone Bug PRs that fix a bug with a specific milestone labels Feb 1, 2022
Copy link
Member

@andrewbranch andrewbranch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like I’ve fixed another crash with the same root cause in the past, which is +1 for the usefulness of fixing the type, but possibly -1 for the ease of fixing the type, because I probably looked into it at the time as well.

@sandersn
Copy link
Member Author

sandersn commented Feb 1, 2022

I'm nearly done changing the type and it feels not very valuable so far. I'll send it out as a separate PR.

@sandersn sandersn merged commit 1ebdcc6 into main Feb 1, 2022
@sandersn sandersn deleted the fix-inlay-hint-jsdoc-function-crash branch February 1, 2022 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Milestone Bug PRs that fix a bug with a specific milestone
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Crash in inlayHints with JSDoc function type
3 participants