-
-
Notifications
You must be signed in to change notification settings - Fork 46
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
completion/blink.cmp: add #510
base: main
Are you sure you want to change the base?
Conversation
32f06ec
to
c2185f2
Compare
4d88661
to
4602327
Compare
This is exactly the completion plugin I was hoping for. 👍 |
c04abc9
to
b06c6c9
Compare
ac1fd11
to
94b0316
Compare
83a2def
to
b41af9d
Compare
blink-cmp = pkgs.callPackage ./packages/blink-cmp.nix { | ||
src = inputs.plugin-blink-cmp; | ||
version = inputs.plugin-blink-cmp.shortRev or inputs.plugin-blink-cmp.shortDirtyRev or "dirty"; | ||
}; |
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.
This might be better suited for the legacyPackages
in the flake-parts compontent, which is propagated into our packages via overlays. See nil (and previously rnix) for how this is done.
providers = mkOption { | ||
type = attrsOf providerType; | ||
default = {}; | ||
description = "Providers"; | ||
}; |
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.
A better description or an example wouldn't be amiss here.
added blink.cmp
why do I get extra backslash sometimes when using lsp?can't repro anymore, prolly fixed upstream- [ ] update script(usedallowBuiltinFetchGit
instead)