Skip to content

fix(blink): don't check blink.cmp markdown config isn't a list#793

Merged
neo451 merged 5 commits into
obsidian-nvim:mainfrom
antinomie8:main
Apr 8, 2026
Merged

fix(blink): don't check blink.cmp markdown config isn't a list#793
neo451 merged 5 commits into
obsidian-nvim:mainfrom
antinomie8:main

Conversation

@antinomie8
Copy link
Copy Markdown
Contributor

blink.cmp source injection

blink.cmp supports an inherits_default key in its per_filetype sources, which fails the util.islist check for adding the plugin's completion source, so I removed it. Additionally I updated util.islist so it uses vim.islist in neovim >= 0.12 as the comments said to do.

PR Checklist

  • The PR contains a description of the changes
  • I read the [CONTRIBUTING.md] file
  • The CHANGELOG.md is updated
  • The changes are documented in the README.md file
  • The code complies with make chores (for style, lint, types, and tests)

@neo451
Copy link
Copy Markdown
Member

neo451 commented Apr 8, 2026

the change makes sense, but the util file change is not what the comment meant, it was that after 0.12 we will have minimum 0.11 so that a util/compat stuff are all not required #778 but this will be merged down the line, so remove the thing and I'll merge

@neo451
Copy link
Copy Markdown
Member

neo451 commented Apr 8, 2026

thank you for the fix!

@all-contributors add @antinomie8 for code

@neo451 neo451 merged commit 6a207b0 into obsidian-nvim:main Apr 8, 2026
10 checks passed
@allcontributors
Copy link
Copy Markdown
Contributor

@neo451

I've put up a pull request to add @antinomie8! 🎉

neo451 added a commit that referenced this pull request Apr 18, 2026
* fix(blink): don't check blink.cmp markdown config isn't a list

blink.cmp supports an inherits_default key in its per_filetype sources

* feat(util): use vim.islist for neovim >=0.12

* docs: update changelog

* Revert "feat(util): use vim.islist for neovim >=0.12"

This reverts commit b564654.

---------

Co-authored-by: neo451 <111681693+neo451@users.noreply.github.com>
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