Skip to content

feat: allow specifying the binary to launch neovim in tests#449

Merged
Conni2461 merged 1 commit intonvim-lua:masterfrom
mrcjkb:master
Jan 10, 2023
Merged

feat: allow specifying the binary to launch neovim in tests#449
Conni2461 merged 1 commit intonvim-lua:masterfrom
mrcjkb:master

Conversation

@mrcjkb
Copy link
Copy Markdown
Contributor

@mrcjkb mrcjkb commented Jan 9, 2023

Use case: Run tests for neovim derivations packaged with Nix.

See also:
* NixOS/nixpkgs#78033
* mrcjkb/haskell-tools.nvim#93 (comment)
@Conni2461
Copy link
Copy Markdown
Collaborator

hey a fellow nixos user ^^

thanks :)

@Conni2461 Conni2461 merged commit 4278bcb into nvim-lua:master Jan 10, 2023
@teto
Copy link
Copy Markdown
Contributor

teto commented Jan 10, 2023

haha you were so fast ! I will try to check if generating a binary wrapper of neovim fixes that. It's a bit annoying since I like to peek at the wrapper but it could be optional at first, just to unblock you.

@mrcjkb
Copy link
Copy Markdown
Contributor Author

mrcjkb commented Jan 10, 2023

hey a fellow nixos user ^^

thanks :)

It's always nice to meet fellow NixOS users 😁

@mrcjkb
Copy link
Copy Markdown
Contributor Author

mrcjkb commented Jan 10, 2023

haha you were so fast ! I will try to check if generating a binary wrapper of neovim fixes that. It's a bit annoying since I like to peek at the wrapper but it could be optional at first, just to unblock you.

Not sure I follow what you mean. Generating a binary wrapper instead of a shell script wrapperr in pkgs.wrapNeovim?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants