Skip to content

Conversation

srdtrk
Copy link
Contributor

@srdtrk srdtrk commented May 23, 2024


NOTE
Please verify that the base repository above has the intended destination!
Github by default opens Pull Requests against the parent of a forked repository.
If this is your personal fork and you didn't intend to open a PR for contribution
to the original project then adjust the base repository accordingly.


@srdtrk srdtrk changed the title lint: fix lsp warning lint: fix lsp warning in vim.lsp.inlay_hint.is_enabled May 23, 2024
Co-authored-by: Tom Kuson <mail@tjkuson.me>
@feoh
Copy link
Collaborator

feoh commented Jul 21, 2024

Hi! Can you please help me understand what this change is meant to fix? Where are you seeing this warning?

Thanks for your contribution!

@VlaDexa
Copy link
Contributor

VlaDexa commented Jul 21, 2024

This was a neovim issue with their documentation where they showed that vim.lsp.inlay_hint.is_enabled requires an argument, when it actually doesn't.

Also someone reported that omitting bufnr causes issues, so I included it in #936

@feoh
Copy link
Collaborator

feoh commented Jul 22, 2024

Thanks so much @VlaDexa - merging!

@feoh feoh merged commit 6f3fe35 into nvim-lua:master Jul 22, 2024
@srdtrk srdtrk deleted the serdar/fix-lsp-warning branch July 22, 2024 07:36
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.

4 participants