-
-
Notifications
You must be signed in to change notification settings - Fork 266
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
plugins/wakatime: init #1652
plugins/wakatime: init #1652
Conversation
Note: the |
I'm sure you're aware already, but the cli is available in nixpkgs, we could include that version if there is a way to disable the vim plugin's autodownload. If it's entirely statically linked (including libc, etc) that's not a big deal though. EDIT: |
I agree, we could, but is it worth the hassle ? |
Maybe we could have an option to let the user choose... |
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.
I agree, we could, but is it worth the hassle ?
I have checked, and the plugin does work fine with the auto-install.
If everything works fine then this isn't a major concern for now.
Maybe we could have an option to let the user choose...
Some purists may like to have as much as possible installed in a deterministic way at build time, however that's something we could revisit later.
LGTM!
Definitely ! |
Add support for vim-wakatime.
This module is quite barebone, but at the same time, there is no need to do much.
Fixes #1649