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(dap): load vscode launch files with jsonc parser #1839

Merged
merged 1 commit into from
May 15, 2024

Conversation

moetayuko
Copy link
Contributor

@moetayuko moetayuko commented Oct 22, 2023

This seems to be the proper fix for #1503. jsonc ensures compatibility with native vscode.

Ref: mfussenegger/nvim-dap#964

@sid-6581
Copy link

sid-6581 commented Nov 9, 2023

An even simpler way is using neoconf's bundled jsonc parser, which saves an extra dependency:

require("dap.ext.vscode").json_decode = require("neoconf.json.jsonc").decode_jsonc

This seems to be the proper fix for LazyVim#1503. jsonc ensures compatibility
with native vscode.

Ref: mfussenegger/nvim-dap#964
@moetayuko moetayuko changed the title fix(dap): load vscode launch files with json5 parser fix(dap): load vscode launch files with jsonc parser Feb 13, 2024
@folke folke merged commit 543dead into LazyVim:main May 15, 2024
3 checks passed
@folke
Copy link
Collaborator

folke commented May 15, 2024

Cool, that's indeed a nice way to fix this. Thanks!

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.

None yet

3 participants