-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat(plugins): support avante.nvim #139
Conversation
Signed-off-by: Aaron Pham <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, thanks so much for the addition! Made one comment about the hardcoded values in the config, and the only other thing is to please update the README
file with the new plugin and a link to it's repo!
Signed-off-by: Aaron Pham <[email protected]>
Signed-off-by: Aaron Pham <[email protected]>
Signed-off-by: Aaron Pham <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, looks great. A couple other nitpicks and then it's good to go. I have also pushed to main
the updates to the diff highlight groups so it should be good to go as well
Co-authored-by: Micah Halter <[email protected]>
Co-authored-by: Micah Halter <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks so much again for the contribution!
Signed-off-by: Aaron Pham [email protected]
📑 Description
Add support for yetone/avante.nvim colorscheme. I'm not entirely sure what to do with the diff color here, see yetone/avante.nvim#283.
The current color I have for diff is not in the palette. Would definitely need a second opinion here.