Skip to content

allow raw lua in blink-cmp keymap via lib.nixvim.mkRaw#4293

Open
DasGruene wants to merge 1 commit into
nix-community:mainfrom
DasGruene:fix/blink-cpm_allow_lua_keymap
Open

allow raw lua in blink-cmp keymap via lib.nixvim.mkRaw#4293
DasGruene wants to merge 1 commit into
nix-community:mainfrom
DasGruene:fix/blink-cpm_allow_lua_keymap

Conversation

@DasGruene
Copy link
Copy Markdown

Allows to define raw lua commands to the keymap section of blink-cmp.

as im quite new to coding in nix, im not sure I've used the best approach, please do fell free to come with constructive suggestions.

Note: personally needed this to add code completion through minuet

@nixvim-ci nixvim-ci Bot requested review from Balssh and khaneliman May 14, 2026 16:45

keymap = defaultNullOpts.mkNullableWithRaw' {
type = with types; either (attrsOf anything) (enum [ false ]);
type = with types; either (attrsOf (maybeRaw anything)) (enum [ false ]);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand why this is needed. Surely anything already covers raw lua?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is quit an embarrassing mistake 😅 in my search for a solution I definitely mixed a few things together that that i shouldnt have.

turns out the solution is to "just" use lib.nixvim.mkRaw.

I will remove this, but then there is really only the examples/documentation left, I wil let you decide if there is any worth in that.

But anyway, Thanks for taking the time to look at my mistakes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is really only the examples/documentation left,

Improving documentation and examples is always valuable. In this case having better examples may have avoided your confusion.

When you rebase, could you use our path/scope-style commit messages? See git log -- plugins/by-name/blink-cmp for examples or CONTRIBUTING.md for docs.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saw this a bit too late, but I will do some light clean up and get back to you.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ive cleaned up my commit messages now 😉

@DasGruene DasGruene force-pushed the fix/blink-cpm_allow_lua_keymap branch from 83062ca to 160b2a1 Compare May 14, 2026 19:01
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.

2 participants