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 converting vam.pluginDictionaries to VimL #74846
Conversation
As an example, this is one of the places where I've needed this conversion: config/neovim.nix#L432-L438 |
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.
In that case let's go ahead with this. As I said, I don't know much about vam but @rummik seems to know what they're doing and the diff looks good to me.
Can you prefix your commit message with vim-utils:
?
@timokau Done! 😁 |
Thanks for taking the care for this! Sorry you had to bring so much patience to the table for your first contribution. Its just an area of the code that not many people are comfortable with. Welcome to the team :) |
Understandable! xD Reviewers seem spread pretty thin anyway, so I wasn't really worrying about it 😅 |
And thanks as well! |
Motivation for this change
Fix a regression that broke outputting attributes on VAM's plugin dictionaries. This also makes naming a little more clear, and escapes the strings it outputs, fixing a bug in the conversion
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @jonringer