Skip to content
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

Inlay warning descriptions in completion list #151

Merged
merged 2 commits into from
Feb 8, 2024
Merged

Conversation

mhutch
Copy link
Owner

@mhutch mhutch commented Feb 8, 2024

This makes it easier to browse the list, as the descriptions are easier to browse than the code.

image

Note that it does not yet support filtering on the description as simply setting the filter text does not result in good matching behavior.

@mhutch mhutch force-pushed the warning-descriptions branch 3 times, most recently from 0ba29cc to c33eeef Compare February 8, 2024 05:32
* Edit out the formatting holes
* Edit to frontload info so they can better be ellipsiszed for
  completion hints
* Use short sentences where possible so they can be cleanly truncated
  on sentence boundaries for completion hints
This makes it easier to browse the list, as the warning descriptions
are easier to browse than the warning codes.

Note that it does not yet support filtering on the description as
simply setting the filter text does not result in good matching behavior.
@mhutch mhutch merged commit c6127b1 into main Feb 8, 2024
1 check passed
@mhutch mhutch mentioned this pull request Mar 28, 2024
@mhutch mhutch deleted the warning-descriptions branch April 30, 2024 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant