Skip to content

fix(docs): fix syntax error in Lua snippet for vim.lsp.document_color #33612

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

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

theHamsta
Copy link
Member

9ff1239 added support for 'textDocument/documentColor' but the text snippet in the Lua docs seem to contain a syntax error.

@gpanders
Copy link
Member

Can you please make the change in the corresponding Lua source file and then regenerate the help docs with make doc?

@theHamsta theHamsta force-pushed the documentColor-then branch from 69f329a to db9d208 Compare April 24, 2025 19:00
@theHamsta
Copy link
Member Author

Can you please make the change in the corresponding Lua source file and then regenerate the help docs with make doc?

Sorry, should be fixed now.

@MariaSolOs
Copy link
Member

Also to fix the commit linter use fix(docs) instead of docs(lsp)

9ff1239 added support for 'textDocument/documentColor' but the
text snippet in the Lua docs seem to contain a syntax error.
@theHamsta theHamsta force-pushed the documentColor-then branch from db9d208 to c45366c Compare April 24, 2025 19:01
@theHamsta theHamsta changed the title docs(lsp): fix syntax error in Lua snippet for vim.lsp.document_color fix(docs): fix syntax error in Lua snippet for vim.lsp.document_color Apr 24, 2025
@justinmk justinmk enabled auto-merge (squash) April 24, 2025 19:07
@justinmk justinmk merged commit 1670fbe into neovim:master Apr 24, 2025
30 checks passed
brynne8 pushed a commit to brynne8/neovim that referenced this pull request Apr 25, 2025
…neovim#33612

9ff1239 added support for 'textDocument/documentColor' but the
text snippet in the Lua docs seem to contain a syntax error.
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.

None yet

4 participants