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
vimPlugins.coc-nvim: 0.0.72 -> 0.0.73 (and change override) #64749
Conversation
cc: @teto, @rvolosatovs |
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've noticed the runtime going out of sync with the javascript causing problems so I support this.
Now that the script is fully described in overrides.nix, maybe we should remove it from vim-plugin-names ?.
@timokau
pkgs/misc/vim-plugins/overrides.nix
Outdated
# The previous fix for this added the index.js file in for a given release, but this caused | ||
# issues since when vimPlugins was auto-updated, the override wasn't always updated, resulting in | ||
# the index.js file being out of sync with the rest of the plugin. This override solves that | ||
# problem by completely overriding the plugin with a release version. |
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.
We don't need this much comment. One can use git blame to see a full explanation.
You could narrow it down to "only official releases contains the required index.js file"
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.
Changed.
I have no problem with that if it makes things easier. I don't (yet) use coc myself. |
Alright, I've edited the comment at @teto's request, removed |
Hi @malob @teto, sorry for the desperate call in this PR but this is all I found on my current (related) issue and I was wondering if you could point me in the right direction? I don't understand all the NodeJS stuff and have always struggled understanding how I upgraded to the latest NixOS unstable and I am now getting this error:
This happens when I use Click here to see the full logs
Maintainers have suggested upgrading to the latest Appreciate any hints, thanks! 🙏🏽 |
@gvolpe, I don't use Coc.nvim anymore since I upgraded to Nvim 0.5, so I'm not very familiar with how Coc related stuff is handled in
So looks like updating should just require running the |
Ah thanks for your response @malob ! I've been postponing the migration to the native LSP stuff for a while, enough yaks on my shaving list already 😄 I'll make sure to check on that PR, thanks a lot, have a nice day! |
In case anyone comes across the same issue, an overlay fixed it for me: gvolpe/nix-config@badd9a4 |
Motivation for this change
coc.nvim
requires abuild/index.js
file that's only available in the release version. As such, auto-updating isn't possible with the current script since master doesn't contain this file.The previous fix for this added the
index.js
file in for a given release, but this caused issues since when vimPlugins was auto-updated, the override wasn't always updated, resulting in theindex.js
file being out of sync with the rest of the plugin.This override solves that problem by completely overriding the plugin with a release version.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)