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

Update plugin code to work with Lazy.nvim and on nvim 0.10.0 #13

Draft
wants to merge 31 commits into
base: main
Choose a base branch
from

Conversation

jondkinney
Copy link

@jondkinney jondkinney commented Jul 29, 2024

95% of this code was written with aider, itself. This branch is currently working for me, but I intend to go through and clean up some of the code/commits before it'd be mergable.

I'm also curious about feedback from @joshuavial on whether or not he agrees with some of the liberty I took and changes I made to things like the default keybindings as well as using user commands instead of relying on calling through :lua ..., etc.

…lled through the `:lua` command and allowed the `:AiderOpen` command to be run directly.
…n fails while trying to open Aider using the AiderOpen command.
…n to resolve "attempt to call global 'is_valid_buffer' (a nil value)" error.
…p level being "Aider" and the rest of the commands having reasonable names. Also changed the space after leader to be a capital A.
…or using the AiderOpen and AiderBackground commands.
…dModifiedFiles` function directly callable by the user in command mode.
Replaced `wk.register` with `wk.add` in `lua/keybindings.lua`.

chore(main): update wk.add usage to match documentation

Restructured the keybindings in the `lua/keybindings.lua` file to resolve the "Invalid field" errors.

Updated the keybindings to use wk.add()

Added new keybindings for Aider (GPT-3.5) and Git modified files in the keybindings.lua file.

Updated the `wk.add()` command to use the new format.

Reformatted the Lua code and added new keybindings for Aider functionality.
@jondkinney jondkinney marked this pull request as draft July 29, 2024 15:47
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.

1 participant