-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
mkdir.nvim, nix-develop.nvim, direnv.vim: init, blink-cmp: add source options, friendly-snippets source #667
base: main
Are you sure you want to change the base?
Conversation
40ea462
to
5d89f9d
Compare
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.
LGTM.
CC @horriblename for blink-cmp side of things.
Some blink options moved around when it was updated so I'm not sure if this will work correctly if merged right now (as blink itself does not work on main, so I can't test how it will behave with the newer version once blink is fixed). If no one else gets to it for a few days I might try and fix it. |
53d0e79 should fix blink on main, although I don't quite have it set up so I had to push the change with a quick dev configuration instead of an actual configuration. |
I'll give it a try on my end, hopefully will still work/just require most recent blink-ripgrep-nvim instead of the version I used. |
Use a submodule to allow arbitrary source additions. Automatically load sources that are enabled. Automatically add enabled sources via blink-cmp setupOpts. Prepopulate submodule with a few basic sources, disabled by default.
Just adds it to the environment. blink-cmp will pick up on it automatically.
Also fixes a misaligned contributor name.
5d89f9d
to
0dda8f6
Compare
Works as expected, did still update blink-ripgrep.nvim as it works with the version of blink.cmp we have now (as well as rebasing on newest main). |
Adds a few very simple plugins I use, as well as options for blink-cmp that I find useful.
I can split this into a couple PRs if that would be preferred.
Two blink-cmp options:
blink-cmp.sourcePlugins
, for arbitrary blink-cmp completion sourcesI'm not sure how to indicate in the docs that they are present without making them
disappear as soon as any other sources are added.
blink-cmp.friendly-snippets.enable
, for an alternate snippet sourcehandling as other options
I also snuck in some typo fixing in the release notes (in a separate commit).
Sanity Checking
nix fmt
).#nix
(default package).#maximal
.#docs-html
(manual, must build).#docs-linkcheck
(optional, please build if adding links)x86_64-linux
aarch64-linux
x86_64-darwin
aarch64-darwin
Add a 👍 reaction to pull requests you find important.