Make health checks more considerate #1709
notfirefox
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
For reference, this is my current
blink.cmp
config:This is the
:checkhealth
output that I get:Two thinks that I would like to object here.
lua
implementation of the fuzzy matcher, I am clearly not concerned whether my system is supported by the pre-built binaries and whetherblink_cmp_fuzzy
is downloaded/built. I think it would be great if we could skip such steps if the user intentionally does not use the rust fuzzy matcher implementation.cmdline
option and yet I am warned about the fact that thecmdline
source may be loaded dynamically. The warning also isn't very clear to me. Is it justcmdline
that may be enabled dynamically, or does this affect other sources as well. And if it is justcmdline
, wouldn't it make sense to silence the warning when disabling it?So as you can tell, those are mostly nitpicks. My impression overall has been largely positive with
blink.cmp
and I have been able to switch fromnvim-cmp
to it without missing much. But as I was revising my Neovim config and checked the:checkhealth
output, this is just something I came across of. I thought I would drop this as some sort of feedback here.Beta Was this translation helpful? Give feedback.
All reactions