-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Install wakatime-cli binary if missing #5
base: main
Are you sure you want to change the base?
Conversation
00e3d9f
to
2364037
Compare
Client plugins should install wakatime-cli themselves, see https://wakatime.com/help/creating-plugin#plugin-init:wakatime-cli
2364037
to
f8ea2d2
Compare
Hi! Just checking in, have you had the time to look at this? If you don't thats fine, ill release a first version of zed wakatime without this merged in ;) |
Hi ewen, I have read Wakatime guidelines for plugins. This was more of a design decision than just straight not implementing it. I personally prefer to manage myself upgrades through declarative configuration. This said, I am more than open to dicussion. I see how this could refrain users. I see two solutions:
Concerning the PR, a few comments:
Bonne journée |
Client plugins should install wakatime-cli themselves, see
https://wakatime.com/help/creating-plugin#plugin-init:wakatime-cli