Skip to content

Conversation

@kiyoon
Copy link
Collaborator

@kiyoon kiyoon commented Dec 11, 2025

  1. vim.g.no_python_maps or vim.g.no_plugin_maps to avoid conflicts. This actually got me because it seems totally working but actually is broken (like repeatable move etc. won't work)
    • A better way is to overwrite the buffer mappings, but it will be too difficult to explain in the main docs unless we actually provide a convenience function that helps mapping.
  2. nvim-next doesn't support main branch of this plugin, thus removed.
  3. I think it's unintuitive that "move function" uses ]m and "select function" uses af, if. The standard seems to be using am and im for selecting function as per https://github.com/neovim/neovim/blob/master/runtime/ftplugin/ruby.vim . I know this is just an example, but keeping the main doc following the common convention helps a lot.

@kiyoon kiyoon requested a review from clason December 11, 2025 02:24
@clason clason force-pushed the docs-no-plugin-maps branch from f112164 to 25319d3 Compare December 13, 2025 13:46
@clason
Copy link
Collaborator

clason commented Dec 13, 2025

CI is fixed. (Do you want to squash your commits?)

@kiyoon kiyoon merged commit 0d7c800 into main Dec 14, 2025
3 checks passed
@kiyoon kiyoon deleted the docs-no-plugin-maps branch December 14, 2025 00:34
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