-
-
Notifications
You must be signed in to change notification settings - Fork 295
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
plugins/lazydev: init #2774
base: main
Are you sure you want to change the base?
plugins/lazydev: init #2774
Conversation
105f64b
to
0896a2d
Compare
28dd237
to
5192494
Compare
&& !config.plugins.cmp.enable | ||
&& cfg.settings.integrations.cmp | ||
) | ||
[ "Nixvim(plugins.lazydev): you have enabled nvim-cmp integration but plugins.cmp is not enabled." ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[ "Nixvim(plugins.lazydev): you have enabled nvim-cmp integration but plugins.cmp is not enabled." ] | |
[ "Nixvim (plugins.lazydev): you have enabled nvim-cmp integration but plugins.cmp is not enabled." ] |
I think we (mostly) put a space there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least on blink-cmp there is not:
Nixvim(plugins.blink): `settings.documentation` does not correspond to a known setting, use `settings.windows.documentation` instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know which formatting we prefer, but we're definitely not very consistent about it currently! 🙈
I'll hold my hands up to contributing some of the inconsistency on this, too.
If we really want to be consistent, we probably need some systematic method of achieving that...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes...
5192494
to
605560a
Compare
605560a
to
11fdc75
Compare
Adds lazydev.nvim to nixvim