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
nodePackages.vim-language-server: init at 2.1.0 #91142
Conversation
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.
@leungbk Nice addition! I was just looking to use this myself and noticed it was missing.
The changes (besides conflicts) look good to me. I built it from your branch and tried it out in vim with LanguageClient-neovim - let g:LanguageClient_serverCommands.vim = ['/nix/store/.../vim-language-server', '--stdio']
- and it works :-)
@calbrecht I haven't reviewed any PRs yet in nixpkgs, but I'd like to start. Since you are the suggested reviewer, is there anything else to be checked usually? I did read the relevant documentation, but I'm not clear on how one should review changes in node-packages.nix
or similar, to ensure that the appropriate generated changes are present and haven't been tinkered with or done erroneously.
470cc04
to
8e19268
Compare
@matt-snider although beeing suggested as reviewer, i did not do so much reviews already. But i know there is this chapter about reviewing in nixpkgs manual. |
@calbrecht Thanks for the response. I think the issue I mentioned with @leungbk It looks good to me, and like I said, it works, but I don't have merge rights for this repo, so for now I can just review and wait for someone else to merge it |
8e19268
to
c0541c1
Compare
c0541c1
to
1c3feb1
Compare
1c3feb1
to
9624953
Compare
9624953
to
8d4cc56
Compare
Motivation for this change
Add vim-language-server, a LSP-compliant server for Vimscript
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)