Skip to content

feat(mappings): false or empty string to disable mappings#192

Open
Kaiser-Yang wants to merge 1 commit into
igorlfs:mainfrom
Kaiser-Yang:main
Open

feat(mappings): false or empty string to disable mappings#192
Kaiser-Yang wants to merge 1 commit into
igorlfs:mainfrom
Kaiser-Yang:main

Conversation

@Kaiser-Yang
Copy link
Copy Markdown

No description provided.

@igorlfs
Copy link
Copy Markdown
Owner

igorlfs commented May 24, 2026

This can be achieved with empty tables. Why should we offer alternatives?

@Kaiser-Yang
Copy link
Copy Markdown
Author

Kaiser-Yang commented May 24, 2026

This can be achieved with empty tables. Why should we offer alternatives?

yeah, you are right. I just forgot this. But this PR seems not too bad.

@igorlfs
Copy link
Copy Markdown
Owner

igorlfs commented May 26, 2026

I just forgot this.

I'll update the docs to make this clearer

But this PR seems not too bad.

If we were to allow the usage of booleans as a shorthand for disabling, we'd also have to update the validation from lua/dap-view/setup/validate/keymaps.lua (which is not a big deal by itself). My point is more so that this has a really marginal benefit, regardless of the effort.

igorlfs added a commit that referenced this pull request May 26, 2026
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