Skip to content
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

fix(completion): validation of process_items #1648

Closed
wants to merge 1 commit into from

Conversation

Lattay
Copy link
Contributor

@Lattay Lattay commented Mar 7, 2025

This patch addresses a recent regression where an error would be raised if lsp_completion.process_items was set.
The validation now behaves as expected: it raises if lsp_completion.process_items is set to something that is not a function.

Commit b8f6278 introduced a regression where it raise an error if
lsp_completion.process_items is set in user config.
@echasnovski
Copy link
Owner

There is always something obvious missing :( And this time it is a big one. Thanks for noticing and the PR! Should be resolved on latest main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants