Skip to content
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

doc: follow up on #141: VADER_OUTPUT_FILE #142

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

blueyed
Copy link
Collaborator

@blueyed blueyed commented Oct 27, 2017

No description provided.

@rdnetto
Copy link
Contributor

rdnetto commented Oct 29, 2017

Nit: VADER_OUTPUT_FILE=/dev/stderr doesn't actually work with Neovim - you need to write it to a file and then cat it afterwards. It might be simpler to just say it's useful with Neovim because the test results are not logged to the terminal by default.

@blueyed
Copy link
Collaborator Author

blueyed commented Oct 29, 2017

OK, will improve it more.
However, I am using this with Neovim.

How do you run Vader with Neovim?

@rdnetto
Copy link
Contributor

rdnetto commented Oct 30, 2017

env VADER_OUTPUT_FILE=/dev/stderr nvim -N -u vimrc.vim -c 'Vader! **/*.vader'                                                                                                           

Doesn't work for me, where vimrc.vim is a minimal vimrc used in CI.

@blueyed
Copy link
Collaborator Author

blueyed commented Mar 25, 2018

Neovim needs the --headless option additionally, or stdout needs to be redirected (so that it's not a TTY).

With Neomake I am using basically this: VADER_OUTPUT_FILE=/dev/stderr nvim --headless -Nu vimrc -c 'Vader! *' 2>&1 </dev/null >/dev/null.

It works on Travis for Vader itself without any special handling (except for setting VADER_OUTPUT_FILE), likely because it's not a tty altogether there.

Ref: neovim/neovim#4772

@blueyed
Copy link
Collaborator Author

blueyed commented Aug 10, 2018

Worked on this part a bit: #175

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.

2 participants