Skip to content

add skip_comments=true to neovim setup#2822

Closed
khyperia wants to merge 1 commit intorust-lang:mainfrom
khyperia:skip_comments
Closed

add skip_comments=true to neovim setup#2822
khyperia wants to merge 1 commit intorust-lang:mainfrom
khyperia:skip_comments

Conversation

@khyperia
Copy link
Copy Markdown
Contributor

@khyperia khyperia commented Apr 5, 2026

the suggested neovim setup reads src/etc/rust_analyzer_zed.json

comments were added to zed's json a month ago, in rust-lang/rust@6d6493a

neovim's vim.json.decode is spec-conformant and rejects comments as invalid json by default

thankfully, in neovim 0.12 (released a week ago), a new option was added to vim.json.decode, { skip_comments = true }, which allows parsing javascript-style comments in json

yippee!

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 5, 2026

Thanks for the PR. If you have write access, feel free to merge this PR if it does not need reviews. You can request a review using r? rustc-dev-guide or r? <username>.

@rustbot rustbot added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Apr 5, 2026
@khyperia
Copy link
Copy Markdown
Contributor Author

khyperia commented Apr 5, 2026

ah, wait, this conflicts with #2814 which is a major rewrite of the neovim section

@khyperia
Copy link
Copy Markdown
Contributor Author

khyperia commented Apr 6, 2026

#2814 has incorporated this change, so I'm closing this - thank you @erfanio !

@khyperia khyperia closed this Apr 6, 2026
@rustbot rustbot removed the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Apr 6, 2026
@khyperia khyperia deleted the skip_comments branch April 6, 2026 04:41
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